Closed
Bug 1013654
Opened 11 years ago
Closed 8 years ago
Make it possible to sample the colour of the menu bar or tabs toolbar.
Categories
(Core :: Widget: Win32, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: mconley, Unassigned, Mentored)
References
Details
(Whiteboard: [lang=c++])
We ran into an issue in bug 940393 where under certain conditions, the tab titles were very hard to read on Windows 8.
We ended up using a pretty hacky approach to read Windows registry entries in order to determine the toolbar colour. It'd be great if we had some kind of platform support so we could tear that hack out.
Updated•10 years ago
|
Mentor: jmathies, tabraldes, netzen, gijskruitbosch+bugs
Whiteboard: [good first bug][lang=c++]
Comment 1•10 years ago
|
||
I would like to work on this bug. Can somebody give me few pointers on what need to be done here?
Updated•10 years ago
|
Flags: needinfo?(tabraldes)
Flags: needinfo?(netzen)
Flags: needinfo?(jmathies)
Flags: needinfo?(gijskruitbosch+bugs)
Comment 2•10 years ago
|
||
This bug will involve replacing http://mxr.mozilla.org/mozilla-central/source/browser/modules/Windows8WindowFrameColor.jsm with more appropriate APIs to determine the titlebar colors. I don't know what those APIs would be - I'm guessing Jim can provide more info here.
Flags: needinfo?(gijskruitbosch+bugs)
Comment 3•10 years ago
|
||
There's some discussion of exposing this through css in bug 578780. We ended up just doing some registry hacks in front end code. Honestly I'd be comfortable resolving this wontfix if the work in bug 940393 is sufficient. Otherwise we should revisit bug 578780.
Flags: needinfo?(jmathies)
Comment 5•10 years ago
|
||
I would prefer that we revisit bug 578780 and close this as wontfix.
Updated•10 years ago
|
Flags: needinfo?(netzen)
Comment 6•10 years ago
|
||
If there's controversy about implementation, this is not a good first bug.
Whiteboard: [good first bug][lang=c++] → [lang=c++]
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•