Closed Bug 1013654 Opened 10 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)

x86_64
Windows 7
defect
Not set
normal

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.
Mentor: jmathies, tabraldes, netzen, gijskruitbosch+bugs
Whiteboard: [good first bug][lang=c++]
I would like to work on this bug. Can somebody give me few pointers on what need to be done here?
Flags: needinfo?(tabraldes)
Flags: needinfo?(netzen)
Flags: needinfo?(jmathies)
Flags: needinfo?(gijskruitbosch+bugs)
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)
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)
Clearing needinfo and deferring to jimm
Flags: needinfo?(tabraldes)
I would prefer that we revisit bug 578780 and close this as wontfix.
Flags: needinfo?(netzen)
If there's controversy about implementation, this is not a good first bug.
Whiteboard: [good first bug][lang=c++] → [lang=c++]
Hi Jim. I am interested to resolve this bug.May be I can help you.
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.