Closed Bug 1163184 Opened 6 years ago Closed 6 years ago
Window titlebar has strange borders in devedition theme on windows
Screenshot : http://cl.ly/image/3F2c0U44442h/o (sidebar thing is unrelated, it's an addon)
The borders are caused by some lw-theme code : - The side border are set on #navigator-toolbox - The top border is set on #browser-panel
Thanks for the report Tim. Any chance could put together a patch that fixes this?
Assignee: nobody → ntim.bugs
Status: NEW → ASSIGNED
Attachment #8603626 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8603626 [details] [diff] [review] Patch Review of attachment 8603626 [details] [diff] [review]: ----------------------------------------------------------------- I don't understand why these need to be !important. I also don't think removing the top border is a good idea. Without the border-top, there is no top border on the window frame, which will look broken. What's more, just removing border-top will only work on aero and win8/10. There's sprinklings of other CSS around that use some background image manipulation tricks to have something resembling a native top border. If the top border looks wrong on win10/win8, we should fix the top border generally, not just remove it in the devedition case.
Attachment #8603626 - Flags: review?(gijskruitbosch+bugs) → review-
Whiteboard: [devedition-40][difficulty=easy] → [polish-backlog][difficulty=easy]
Split this off from the patch in bug 1188194.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
No longer depends on: 1188194
Approval Request Comment [Feature/regressing bug #]: devedition theme look&feel on windows [User impact if declined]: borders on devedition that it doesn't need [Describe test coverage new/current, TreeHerder]: nope, styling only [Risks and why]: low/none, devedition-theme-CSS-only change [String/UUID change made/needed]: nope
Comment on attachment 8654160 [details] [diff] [review] Patch v2 Polish, taking it.
Attachment #8654160 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I have reproduced this bug on Firefox Version 42.0a1 It's fixed and verified on Latest Firefox Beta & DevEdition Build ID 20151005144425 User Agent Mozilla/5.0 (Windows NT 6.3; rv:42.0) Gecko/20100101 Firefox/42.0 Latest DevEdition Build ID 20151008004031 User Agent Mozilla/5.0 (Windows NT 6.3; rv:43.0) Gecko/20100101 Firefox/43.0 Tested OS- Windows8.1 32bit
You need to log in before you can comment on or make changes to this bug.