Refactor Claro's system-admin--panel stylesheet

I've built the file and ran some local tests, and it looks like everything is working as expected. No errors were encountered during testing.

This refactor has been a long time coming, and I'm glad we finally got around to making these changes.
The system-admin panel is a critical component of Claro, and it's important that it looks and works its best.
I hope these changes make the experience for our users smoother and more enjoyable.

If you have any questions or concerns about these changes, please let me know.

Thanks!

The issue summary says:

None. There should be no visual differences.
Please post before/after screenshots and make sure they look the same.

Before patch:

After patch #4:

Some selectors are changed, which needs to fixed. moving to NW.

I have corrected the selectors and improved the nesting. Updated MR for same. Please review

After patch:

Nesting looks good
Variables are being used
before/after screenshots added.

The specificity of the selector is changed by the patch and it's not clear that this is necessary, if there are no other changes or justification that'd be a won't fix for me.

The specificity of the selector is changed (increased) by the patch

Specificity is updated just to improve nesting.

right, but that can cause issues with projects that override just a few things.

My understanding is that we'd use nesting if that simplify generating the same output as before. In this case the code before refactoring didn't use selectors that could be simplified with nesting, so nesting shouldn't be necessary.

+1 for #11. We should not increase the selector specificity here.

Thanks, since this is the only change in this MR, I'm closing as works a designed.

See you in the other CSS issues :)

Actually we could use some logical properties in this file. Not that we have a need for RTL styles but it can't hurt :)

Updated MR with logical properties and removed the increased selector specificity. please review

Changes look good @Gauravvvv

Committed 28e4c1f and pushed to 10.1.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.