Closed Bug 882948 Opened 11 years ago Closed 11 years ago

Add widget level support for AeroLite

Categories

(Core :: Widget: Win32, defect)

x86_64
Windows 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

Windows Server 2012 and 2013's theme is AeroLite, and we currently fail to recognize it properly.
Attached patch Patch (v1)Splinter Review
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #762355 - Flags: review?(roc)
Don't forget about media queries in nsMediaFeatures.cpp -

http://mxr.mozilla.org/mozilla-central/source/layout/style/nsMediaFeatures.cpp#31
(In reply to Jim Mathies [:jimm] from comment #2)
> Don't forget about media queries in nsMediaFeatures.cpp -
> 
> http://mxr.mozilla.org/mozilla-central/source/layout/style/nsMediaFeatures.
> cpp#31

Argh! https://hg.mozilla.org/integration/mozilla-inbound/rev/2501f979d629
The patches use both "lite" and "light". I think we should use "lite" consistently to match what Windows calls the theme unless there is a good reason for this.
Flags: needinfo?(ehsan)
(In reply to Matthew N. [:MattN] from comment #5)
> The patches use both "lite" and "light". I think we should use "lite"
> consistently to match what Windows calls the theme unless there is a good
> reason for this.

Embarrassing: https://hg.mozilla.org/integration/mozilla-inbound/rev/5e5abccab73e
Flags: needinfo?(ehsan)
Forgot the other half that actually matters:

https://hg.mozilla.org/integration/mozilla-inbound/rev/0004b0e5b946

:(
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: