STR:
Open DevTools -> Storage Inspector

Expected:

• "add" and "refresh" icons are visible on the buttons in the top bar

Actual:

• The buttons are visible, but the icons aren't

These are the buttons in the markup: https://searchfox.org/mozilla-central/rev/444ee13e14fe30451651c0f62b3979c76766ada4/devtools/client/storage/index.xul#50-53.

And they are styled with CSS and ::before: https://searchfox.org/mozilla-central/rev/444ee13e14fe30451651c0f62b3979c76766ada4/devtools/client/themes/storage.css#73-82.

Tentatively marking this as regressed by Bug 1538983 since it seems the most likely culprit.

It's because this rule doesn't match anymore .devtools-button:not(:empty) since the content isn't anonymous anymore: https://searchfox.org/mozilla-central/rev/444ee13e14fe30451651c0f62b3979c76766ada4/devtools/client/themes/common.css#377 / https://searchfox.org/mozilla-central/search?q=.devtools-button%3Anot%28%3Aempty%29&path=.

I wonder if we still need to support non-empty .devtools-button elements anymore. Belén, do you know?

I think these are the only <xul:button> devtools-button elements so perhaps we can directly switch these to html:button which would also fix it.

(In reply to Brian Grinstead [:bgrins] from comment #1)

It's because this rule doesn't match anymore .devtools-button:not(:empty) since the content isn't anonymous anymore: https://searchfox.org/mozilla-central/rev/444ee13e14fe30451651c0f62b3979c76766ada4/devtools/client/themes/common.css#377 / https://searchfox.org/mozilla-central/search?q=.devtools-button%3Anot%28%3Aempty%29&path=.

I wonder if we still need to support non-empty .devtools-button elements anymore. Belén, do you know?

Yeah, here's a list of places where we use them:

• netmonitor filter buttons
• console filter buttons
• perf tool empty notice
• memory tool empty notice

I think it would make sense to probably split .devtools-button into .devtools-text-button and .devtools-icon-button

We also have .devtools-toolbarbutton which is defined for XUL toolbarbuttons.

https://hg.mozilla.org/mozilla-central/rev/f41ea9d40a05

(sorry :bgrins, was on PTO when this was requested and forgot to update my status on bugzilla)