Closed
Bug 1192276
Opened 8 years ago
Closed 8 years ago
Restore about:privatebrowsing UI to before it was messed up by Project Chameleon styles
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(seamonkey2.35 wontfix, seamonkey2.36 wontfix, seamonkey2.37 fixed, seamonkey2.38 fixed, seamonkey2.39 fixed, seamonkey2.40 fixed)
RESOLVED
FIXED
seamonkey2.40
People
(Reporter: philip.chee, Assigned: philip.chee)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(1 file)
25.07 KB,
patch
|
neil
:
review+
stefanh
:
ui-review+
alfredkayser
:
feedback+
iannbugzilla
:
approval-comm-aurora+
iannbugzilla
:
approval-comm-beta+
iannbugzilla
:
approval-comm-release+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1133380 +++ > Bug 1125636 changed the theming of about:config which we were borrowing from.
![]() |
Assignee | |
Comment 1•8 years ago
|
||
Restore the UI to before Project Chameleon 1. Revert XUL 1.1. Stop using in-content/info-pages.css and global/config.css 2. Copy styles that we need from the old config.css to aboutPrivateBrowsing.css 3. Copy PNGs from toolkit that we were using - some of which don't exist any more. I managed to find them in mozilla-release. The result is that we don't depend on anything in toolkit theme for about:privatebrowsing
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #8645000 -
Flags: ui-review?(stefanh)
Attachment #8645000 -
Flags: review?(neil)
Comment 2•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. I won't be able to look at this until after August 25. Karsten might be an option if you're in a hurry.
status-firefox42:
affected → ---
status-seamonkey2.35:
--- → affected
status-seamonkey2.36:
--- → affected
Comment 3•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. Which version of aboutPrivateBrowsing.xul will third-party themes prefer? Should Modern @import its config.css to avoid the duplication? Speaking of which, are there plans to revert the look of about:config?
Attachment #8645000 -
Flags: feedback?(kairo)
Attachment #8645000 -
Flags: feedback?(alfredkayser)
Comment 4•8 years ago
|
||
I am ok with this change. This will make seamonkey less dependent on all the changes that are being done for Firefox (which I also don't like that much...)
Updated•8 years ago
|
Attachment #8645000 -
Flags: feedback?(alfredkayser) → feedback+
![]() |
Assignee | |
Comment 5•8 years ago
|
||
(In reply to neil@parkwaycc.co.uk from comment #3) > Comment on attachment 8645000 [details] [diff] [review] > Patch v1.0 Proposed fix. > > Which version of aboutPrivateBrowsing.xul will third-party themes prefer? I suspect the reverted version in my patch. I could ask the in the Theme Dev forum in Mozillazine. > Should Modern @import its config.css to avoid the duplication? I think misusing config.css to style the private browsing was sub-optimal. In Modern we could use in-content/info-pages.css if you want to avoid duplication. Plus in classic we override the toolkit info-pages.css > Speaking of which, are there plans to revert the look of about:config? It's on my TODO list but not exactly at the top of of the list. If someone else wants to do this, welcome.
![]() |
Assignee | |
Comment 6•8 years ago
|
||
I've asked for comments from third party themers in Mozillazine: http://forums.mozillazine.org/viewtopic.php?f=18&t=2954291
Comment 7•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. I tested this on Windows, Mac and Linux, and it seems to be fine. You even restored an underline which had gone missing in the Modern theme!
Attachment #8645000 -
Flags: review?(neil)
Attachment #8645000 -
Flags: review+
Attachment #8645000 -
Flags: feedback?(kairo)
![]() |
||
Comment 8•8 years ago
|
||
FWIW, I prefer us keeping everything as close to Firefox as possible, as I'm following that with my theming work (also has the upside of actually releasing builds to the public), but as long as Firefox and SeaMonkey don't expect different content at the same chrome address, I can deal with differences.
![]() |
Assignee | |
Comment 9•8 years ago
|
||
Patrick Dempsey agrees with Alfred Kayser.
Comment 10•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. Yeah, this looks now like current release.
Attachment #8645000 -
Flags: ui-review?(stefanh) → ui-review+
![]() |
Assignee | |
Comment 11•8 years ago
|
||
http://hg.mozilla.org/comm-central/rev/87e3cb843f65
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-seamonkey2.38:
--- → affected
status-seamonkey2.39:
--- → affected
status-seamonkey2.40:
--- → fixed
tracking-seamonkey2.38:
--- → ?
tracking-seamonkey2.39:
--- → ?
tracking-seamonkey2.40:
--- → ?
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.40
![]() |
Assignee | |
Comment 12•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. [Approval Request Comment] Regression caused by (bug #): Bug 1097111 - Update all about: pages to Project Chameleon User impact if declined: awful looking about:privatebrowsing in classic theme. Testing completed (on m-c, etc.): baked on comm-central for a while. Risk to taking this patch (and alternatives if risky): Low risk mostly CSS changes. String changes made by this patch: None
Attachment #8645000 -
Flags: approval-comm-release?
Attachment #8645000 -
Flags: approval-comm-beta?
Attachment #8645000 -
Flags: approval-comm-aurora?
Comment 13•8 years ago
|
||
Comment on attachment 8645000 [details] [diff] [review] Patch v1.0 Proposed fix. a=me
Attachment #8645000 -
Flags: approval-comm-release?
Attachment #8645000 -
Flags: approval-comm-release+
Attachment #8645000 -
Flags: approval-comm-beta?
Attachment #8645000 -
Flags: approval-comm-beta+
Attachment #8645000 -
Flags: approval-comm-aurora?
Attachment #8645000 -
Flags: approval-comm-aurora+
![]() |
Assignee | |
Comment 14•8 years ago
|
||
http://hg.mozilla.org/releases/comm-aurora/rev/0e6fd3bedbfb http://hg.mozilla.org/releases/comm-beta/rev/92570e1ca52e http://hg.mozilla.org/releases/comm-release/rev/83f2af2ad036
status-seamonkey2.37:
--- → fixed
tracking-seamonkey2.38:
? → ---
tracking-seamonkey2.39:
? → ---
tracking-seamonkey2.40:
? → ---
![]() |
||
Comment 15•8 years ago
|
||
(In reply to neil@parkwaycc.co.uk from comment #3) > Should Modern @import its config.css to avoid the duplication? > Speaking of which, are there plans to revert the look of about:config? Is there a bug for reverting about:config appearance in Classic already? Also, about:support and about:addons are other shared Toolkit pages recently redesigned. If there is consensus that the Chameleon style isn't wanted for SeaMonkey it should be consistently reverted (and adaptions are needed for Modern anyway whenever Toolkit makes any layout changes). [I'm commenting here rather than in meta bug 1133743 given that more people are watching this bug.]
![]() |
Assignee | |
Comment 16•8 years ago
|
||
There are two patches that need to land first: 1. comm-central Bug 1022354 Move override for defaultFavicon from content to classic skin now that Bug 1170207 is fixed 2. mozilla central Bug 1189918 - Theme overrides should work in safe mode for comm-central applications too. Once these two bugs land we can safely use override directives in theme manifests without screwing up third party themes.
You need to log in
before you can comment on or make changes to this bug.
Description
•