Skip to content

Adjust todo button margin

Payton Burdette requested to merge pb-fix-todo-padding-issue into master

What does this MR do and why?

In !82236 (merged) we migrated loading icons to follow the design system. Part of this migration left some margin hanging around on the todo buttons.

This MR only adds the margin to the spinner if it is visible in the DOM and removes it once the spinner is not visible.

This was report here in slack https://gitlab.slack.com/archives/CETG54GQ0/p1647364052847649 (internal link)

Screenshots or screen recordings

todo-fix

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Payton Burdette

Merge request reports

Loading