Closed Bug 1514058 Opened 2 years ago Closed 2 years ago

about:debugging sidebar link no longer look selected


(DevTools :: about:debugging, defect, P2)



(firefox-esr60 unaffected, firefox64 unaffected, firefox65+ verified, firefox66+ verified)

Firefox 66
Tracking Status
firefox-esr60 --- unaffected
firefox64 --- unaffected
firefox65 + verified
firefox66 + verified


(Reporter: jdescottes, Assigned: jdescottes)



(Keywords: regression)


(1 file)

Regression from Bug 1512889

From common.css, buttons now force:

  color: inherit !important

Which overrides 

  color: var(--in-content-category-text-selected);

set on `#categories > .category.selected`
Assignee: nobody → jdescottes
Duplicate of this bug: 1512312
This is actually a regression of bug 1473922, since bug 1473922 actually introduced the !important.
Blocks: 1473922
No longer blocks: 1512889
Dão, having seen both patches from bug 1512887 and here, I'm not sure what's the best approach, bug 1512887 seems to be providing a toolkit side fix in common.css, while this bug provides a devtools specific fix in aboutdebugging.css.

I don't have any preference over both approaches since they both have their pros and cons, but I want to make sure we end up having a consistent fix for both bugs. One argument for having the fix in toolkit, is that common.css is supposed to cater for all use-cases. One argument for having the fix in aboutdebugging.css is that about:debugging is the only page that uses buttons as categories, and that we already seem to have some CSS for that:
Flags: needinfo?(dao+bmo)
Doing it all in aboutdebugging.css sounds good to me. Bug 1512887 landed now, and this bug can move the rule added there.
Flags: needinfo?(dao+bmo)
We can do that we just need to make sure to uplift Bug 1512887 first.
Pushed by
Restore color of selected about:debugging category;r=ntim
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Please nominate this for Beta approval when you get a chance (feel free to take the NI in bug 1512887 too if needed).
Flags: needinfo?(jdescottes)
Comment on attachment 9031278 [details]
Bug 1514058 - Restore color of selected about:debugging category;r=ntim

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1473922

User impact if declined: Selected category text color is no longer blue in about:debugging

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: No

Needs manual test from QE?: Yes

If yes, steps to reproduce: Check if selected category text color is blue in about:debugging

List of other uplifts needed: Bug 1512887

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): css only

String changes made/needed: none
Attachment #9031278 - Flags: approval-mozilla-beta?
Flags: needinfo?(jdescottes)
Comment on attachment 9031278 [details]
Bug 1514058 - Restore color of selected about:debugging category;r=ntim

[Triage Comment]
Simple CSS fix, approved for 65.0b5.
Attachment #9031278 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
I’ve reproduced this issue on an affected Nightly build (2018-12-13).
I can confirm the selected about:debugging category turns blue on Beta 65.0b5 (20181217180946) and Nightly 66.0a1 (2018-12-19) builds, with Windows 10 x64, Ubuntu 18.04 x64 and macOS 10.13.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.