Closed
Bug 1133582
Opened 10 years ago
Closed 10 years ago
Warning box in about:config doesn't display properly in Modern any more
Categories
(SeaMonkey :: Themes, defect)
SeaMonkey
Themes
Tracking
(seamonkey2.34 unaffected, seamonkey2.35 fixed, seamonkey2.36 fixed)
RESOLVED
FIXED
seamonkey2.36
Tracking | Status | |
---|---|---|
seamonkey2.34 | --- | unaffected |
seamonkey2.35 | --- | fixed |
seamonkey2.36 | --- | fixed |
People
(Reporter: rsx11m.pub, Assigned: neil)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
3.03 KB,
patch
|
philip.chee
:
review+
iannbugzilla
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1133380 +++
Bug 1125636 changed the theming of about:config in Toolkit, now it's broken in the Modern theme for which config.css needs to be adjusted. Specifically, the box width appears to be no longer constrained, and the warning icon is missing, possibly other issues.
Also, that bug introduced chrome://global/skin/in-content/info-pages.css which we don't have in Modern (no warning is thrown in the error console).
Assignee | ||
Comment 1•10 years ago
|
||
* Moved the max-width from the warningInnerBox to the title/description
* Moved the image from #exclam to the background of the warningBox
* Set the background position of the image to the old padding
* Increased the left padding by the width of the image
* Moved the image's right padding to left padding on the title/description
Assignee | ||
Comment 2•10 years ago
|
||
Fixes the fallout from bug 1133380 too.
Attachment #8565215 -
Attachment is obsolete: true
Attachment #8565215 -
Flags: review?(philip.chee)
Attachment #8567303 -
Flags: review?(philip.chee)
Comment 3•10 years ago
|
||
Comment on attachment 8567303 [details] [diff] [review]
Updated patch
r=me
Some possible tweaks:
> + background-image: url("chrome://global/skin/icons/warning-48.png");
> + background-position: 30px 3em;
> - -moz-padding-start: 30px;
> + -moz-padding-start: 78px;
If you set -moz-padding-start to calc(3em + 48px + 3em)
> -#warningInnerBox
> +.title, .description
> {
> max-width: 50em;
Then you can remove the -moz-padding-start here:
> + -moz-padding-start: 3em;
> -#warningTitle {
> +#privateTitle,
> +#normalTitle {
Then you can remove the -moz-padding-start here:
> + -moz-padding-start: 0px;
> + margin: 0 0 .6em;
> + font-size: 160%;
Attachment #8567303 -
Flags: review?(philip.chee) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.36
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8567303 [details] [diff] [review]
Updated patch
[Approval Request Comment]
Regression caused by (bug #): 1125636
User impact if declined: Ugliness
Testing completed (on m-c, etc.): Landed on c-c
Risk to taking this patch (and alternatives if risky): Low
String changes made by this patch: None
Comment on attachment 8567303 [details] [diff] [review]
Updated patch
Assuming that you meant to request branch approval for this patch, for some reason this didn't quite stick...
Attachment #8567303 -
Flags: approval-comm-aurora?
Attachment #8567303 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Assignee | ||
Comment 7•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•