1133582 - Warning box in about:config doesn't display properly in Modern any more

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[rsx11m]

+++ 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).

Comment 1

[[e-mail]]

Attachment: Proposed patch

* 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

Comment 2

[[e-mail]]

Attachment: Updated patch

Fixes the fallout from bug 1133380 too.

Comment 3

[Philip Chee]

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%;

Comment 4

[[e-mail]]

Pushed comm-central changeset b0d45aade179.

Comment 5

[[e-mail]]

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 6

[rsx11m]

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...

Comment 7

[[e-mail]]

Pushed comm-aurora changeset 360bca7e7026.