561668 - Port |Bug 518989 - Themes cannot give about:support an original design| to Modern

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

about:support and its styles are in Toolkit now. Our Default theme gets those automatically, Modern needs a copy.

Comment 1

[Jens Hatlak (:InvisibleSmiley)]

Looking at Modern's jar.mn I found that it's missing aboutSupport.css as well so I started with bug 547458. Afterwards I'll revert to this one.

Comment 2

[Robert Kaiser]

We'll need this at some point in time of the 2.1 series, but I don't want to block a specific version for it.

Comment 3

[Jens Hatlak (:InvisibleSmiley)]

(In reply to comment #1)
> Looking at Modern's jar.mn I found that it's missing aboutSupport.css as well

s/aboutSupport/aboutMemory/ of course.

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch

about:support is quite similar to about:plugins with respect to the tables so I followed the same approach as in about.css and used the plugins.css import to act as a base. The remaining color definitions are borrowed from about.css and plugins.css.

The first-child rules are needed since plugins.css defines border-top for all tds and aboutSupport.css for th.column, column-wise tables are not used in about:plugins but in about:support, and column-wise ths can be detected but not column-wise tds (I blame the core implementation for that shortcoming) so that rule is the simplest way to achieve the goal, even though its scope is maybe a bit too broad.

@Neil: If there are other possible Modern reviewers please feel free to hand the requests for this bug and bug 547458 off to them. I guess your queue is already long enough.

Comment 5

[[e-mail]]

Comment on attachment 442828 [details] [diff] [review]
patch

>+html {
>+  background: #FFF;
>+}
>+
>+body {
>+  color: #22262F;
>+  font: message-box;
Unnecessary, plugins.css includes this.

>+.page-subtitle {
>+  margin-bottom: 3em;
Too large for my taste. Maybe half as much?

>+th.column {
>+  border-top: 1px dotted #2D3B49;
>+  white-space: nowrap;
>+  width: 0px;
Needs some text-align too, no?

>+.prefs-table {
>+  width: 100%;
Also unnecessary.

>+  table-layout: fixed;
>+}
>+
>+.pref-name {
>+  width: 70%;
>+  white-space: nowrap;
>+  overflow: hidden;
>+}
>+
>+.pref-value {
>+  width: 30%;
>+  white-space: nowrap;
>+  overflow: hidden;
This doesn't actually work, you get a 50/50 split...

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

(In reply to comment #5)
> >+  table-layout: fixed;
> >+}
> >+
> >+.pref-name {
> >+  width: 70%;
> >+  white-space: nowrap;
> >+  overflow: hidden;
> >+}
> >+
> >+.pref-value {
> >+  width: 30%;
> >+  white-space: nowrap;
> >+  overflow: hidden;
> This doesn't actually work, you get a 50/50 split...

Bah, filed bug 563419. Will fix it for Modern right away, though.

Comment 7

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2

(In reply to comment #5)
> >+.page-subtitle {
> >+  margin-bottom: 3em;
> Too large for my taste. Maybe half as much?

If you like. It's only that with the 3em the margins above and below the Copy All to Clipboard button are equal for me (WinXP).

> >+th.column {
> >+  border-top: 1px dotted #2D3B49;
> >+  white-space: nowrap;
> >+  width: 0px;
> Needs some text-align too, no?

Yep. I only saw the th rule in plugins.css but it sets the alignment to center.

Comment 8

[[e-mail]]

(In reply to comment #7)
> It's only that with the 3em the margins above and below the Copy
> All to Clipboard button are equal for me (WinXP).
It's the margins above and below the blurb that I was trying to get equal.

I notice that this page uses HTML buttons, which sucks :-(

Comment 9

[[e-mail]]

Comment on attachment 443161 [details] [diff] [review]
patch v2

>+.major-section {
>+  margin-top: 2em;
>+  margin-bottom: 1em;
>+  font-size: large;
>+  text-align: start;
>+  font-weight: bold;
[As discussed on IRC we can probably lose the font-* styles]

>+tr:first-child th,
>+tr:first-child td {
Use >s please.

>+.prefs-table th.name {
>+  width: 70%;
There is only one element in the document with a class of name, isn't there?

>+  white-space: nowrap;
>+  overflow: hidden;
These styles need to apply to the data cells, not the header.

Comment 10

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v3 [Checkin: comment 11]

cf. bug 563419 comment 1.

Comment 11

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 443210 [details] [diff] [review]
patch v3 [Checkin: comment 11]

http://hg.mozilla.org/comm-central/rev/61a20d4f19f5