Skip to content

Conversation

@dew326
Copy link
Contributor

@dew326 dew326 commented May 15, 2025

🎫 Issue IBX-9940

Related PRs:

Additional PR later:

Description:

For QA:

Documentation:

@sonarqubecloud
Copy link

padding-left: calculateRem(8px);
padding-top: 0;
padding-bottom: 0;
margin-right: calculateRem(8px);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huuge nitpick: moving margin-right to line 114 just so that margins are as a group, just like it's done with paddings.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally, I'm okay with the current order as these lines were not added or modified in this PR, only the indentation has changed. :)

@dew326 dew326 merged commit c0adbf5 into main May 22, 2025
21 of 27 checks passed
@dew326 dew326 deleted the sass-deprecations branch May 22, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants