Closed
Bug 978003
Opened 11 years ago
Closed 11 years ago
Update inContentUI.css for Windows 8 and up
Categories
(Toolkit :: Themes, defect)
Tracking
()
VERIFIED
FIXED
mozilla30
People
(Reporter: Terepin, Assigned: dao)
References
Details
(Whiteboard: [Australis:P4])
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mikedeboer
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release)
Build ID: 20140227030203
Steps to reproduce:
ICUI has still Aero theme, which looks out of place, now even more since bug 960517 landed. This bug is about "about" pages in general. I will fill separate bugs for each "about" page if needed.
Reporter | ||
Updated•11 years ago
|
Assignee | ||
Updated•11 years ago
|
Status: UNCONFIRMED → NEW
Component: Theme → Themes
Ever confirmed: true
Product: Firefox → Toolkit
Summary: In-Content UI (ICUI) themes adjustments for Windows 8 and up → Update inContentUI.css for Windows 8 and up
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8386044 -
Flags: review?(mdeboer)
Comment 2•11 years ago
|
||
Comment on attachment 8386044 [details] [diff] [review]
patch
Review of attachment 8386044 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM. Thanks Dão!
I didn't know about the `-moz-windows-glass` media query (not consciously, at least)... handy!
Attachment #8386044 -
Flags: review?(mdeboer) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 8386044 [details] [diff] [review]
patch
[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 960517
User impact if declined: visual mismatch on Windows 8
Testing completed (on m-c, etc.): landed on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none
Attachment #8386044 -
Flags: approval-mozilla-aurora?
Reporter | ||
Comment 6•11 years ago
|
||
Dao, add-ons list and section selector should be squared and not rounded. Shall I fill bug for it?
Comment 7•11 years ago
|
||
Peter, that is a good idea. This bug is about adjusting the general styles used across in-content pages. The UI element of specific pages are not considered part of this bug. It's going to be a lot of work, but each in-content page will need to be reviewed and adjusted separately. A bug should be filed for each of them.
Comment 8•11 years ago
|
||
(I meant that filing a bug is a good idea)
Reporter | ||
Comment 9•11 years ago
|
||
Is "squaring" a valid English word? :)
Comment 10•11 years ago
|
||
As far as I'm concerned, it is now! :)
Reporter | ||
Comment 11•11 years ago
|
||
Filled. Please, review it and I'll fill rest of the reports accordingly. Thanks.
Updated•11 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Reporter | ||
Comment 12•11 years ago
|
||
Eh, are there plans to restyle about:support?
Updated•11 years ago
|
Attachment #8386044 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 13•11 years ago
|
||
Reporter | ||
Updated•11 years ago
|
Comment 14•11 years ago
|
||
With this bug fixed, the addon buttons (Disable, Remove,...) have the native styling on Windows 8. Not sure if that is expected.
Comment 15•11 years ago
|
||
Dao, can you confirm if Tim's results are expected or not?
Flags: needinfo?(dao)
Assignee | ||
Comment 16•11 years ago
|
||
I think that's expected. Using the native, flat and square button styling is in line the other Win 8 theme adjustments in inContentUI.css and elsewhere.
Flags: needinfo?(dao)
Comment 17•11 years ago
|
||
In that case I'm marking this issue verified. Thank you!
# Fx 29 beta 8: 20140414143035
# Latest Aurora: 20140416004008
You need to log in
before you can comment on or make changes to this bug.
Description
•