1379368 - About pages lost the proper styling in SeaMonkey Classic theme

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[Frank-Rainer Grahl (:frg)]

Bug 1367326 removed some styling options from about.css in toolkit. 

Most SeaMonkey about: pages are now an unappealing gray mess with the classic theme.

Comment 1

[Rainer Bielefeld]

REPRODUCIBLE with unzipped installer of  official en-US SeaMonkey 2.53a1 (NT 6.1; WOW64; rv:56.0) Gecko/20100101 Firefox/56.0 Build 20170706013410  (Default Classic Theme) on German WIN7 64bit.

Was still ok with Build 20170620091523, appeared within 2.53

But FRG already knows all details :-)

Comment 2

[Frank-Rainer Grahl (:frg)]

Attachment: 1379368-aboutcss.patch

Confirmed that Modern is not affected. It already has its own about.css

Comment 3

[rsx11m]

Comment on attachment 8884620 [details] [diff] [review]
1379368-aboutcss.patch

Ok, so we are forking yet another CSS style sheet for the default theme (sigh).

Looks almost identical to prior mozilla-central changeset c92368a3d324, except for:

>+++ b/suite/themes/classic/communicator/about.css
>+ul {
>+  margin: 0;
>+  margin-inline-start: 1.5em;
>+  padding: 0;
>+}

There was a "list-style: square;" in that definition, we don't want that any more?
Most likely there is a reason, I'd just like to know it...

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e85f8a58c5c1
Restore pre Photon styling for SeaMonkey about pages. r=rsx11m

Comment 5

[Frank-Rainer Grahl (:frg)]

> Ok, so we are forking yet another CSS style sheet for the default theme (sigh).

Yeahh. Thought first to do an overlay but with the default sheet looking like <BEEP>, likely needing a ton of changes and only 58 lines I can live with it.

> There was a "list-style: square;" in that definition, we don't want that any more?

ahh oehhh I hmm... Ok, maybe removed in error :) Didn't notice it. The default circles looked ok and almost identical. Thanks for catching it. Added back in the push.

Comment 6

[Frank-Rainer Grahl (:frg)]

Comment on attachment 8884620 [details] [diff] [review]
1379368-aboutcss.patch

[Approval Request Comment]
Regression caused by (bug #):  1367326
User impact if declined: about pages look bland and unappealing.
Testing completed (on m-c, etc.): c-b 
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: --

Comment 7

[Ian Neal]

Comment on attachment 8884620 [details] [diff] [review]
1379368-aboutcss.patch

a=me for c-b

Comment 8

[Frank-Rainer Grahl (:frg)]

https://hg.mozilla.org/releases/comm-beta/rev/656ff307995ad3c250e399571841530c9ad1d4e2