Consolidate Icon CSS Files

[darkwing]

At the moment we have three files which contain CSS for icon display:

• src/components/shared/SourceIcon.css
• src/components/shared/Svg.css
• src/components/PrimaryPanes/Sources.css

There's unnecessary overlap in some of their styles which we should resolve. Ideally we have one CSS file with all of the base icon stuff.

[darkwing]

@notjaril You may enjoy this one 👍

[jaril]

/claim

[claim]

Thanks for claiming the issue! 👋

Here are some links for getting setup, contributing, and developing. We're always happy to answer questions in slack! If you become busy, feel free to /unclaim it.

🦊 Debugger team!

[jasonLaster]

I'm re-opening because I could not land the Accordion.css changes in MC

This commit shows the portion that i had to revert in debugger.css and Accordion.css.
https://hg.mozilla.org/mozilla-central/rev/1c7af47c1ba4

The reason for the revert was that this test (browser_dbg-expressions-focus.js) was failing with it.
The best path forwards is to try to re-land the debugger.css changes in MC.

[darkwing]

@fvsch ^

[fvsch]

(Side note: we should probably update assignee (me) and labels.)

Just to be clear, commit 3d9f98d shows what was reverted too?

[fvsch]

I swear if the issue is that I changed the accordion headers from 24px to 25px height and some clickAtPreciseCoordinatesBecauseWhatCouldGoWrongAmarite(0,25) code is failing, I'm going to shout at some clouds.