The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.1a1

Firefox Tracking Flags

(blocking-seamonkey2.1 needed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
about:support and its styles are in Toolkit now. Our Default theme gets those automatically, Modern needs a copy.
Flags: blocking-seamonkey2.1a1?
(Assignee)

Comment 1

7 years ago
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

7 years ago
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.
blocking-seamonkey2.1: --- → needed
Flags: blocking-seamonkey2.1a1?
(Assignee)

Comment 3

7 years ago
(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.
(Assignee)

Comment 4

7 years ago
Created attachment 442828 [details] [diff] [review]
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.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #442828 - Flags: review?(neil)

Comment 5

7 years ago
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...
(Assignee)

Comment 6

7 years ago
(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.
(Assignee)

Comment 7

7 years ago
Created attachment 443161 [details] [diff] [review]
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.
Attachment #442828 - Attachment is obsolete: true
Attachment #443161 - Flags: review?(neil)
Attachment #442828 - Flags: review?(neil)

Comment 8

7 years ago
(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

7 years ago
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.
Attachment #443161 - Flags: review?(neil) → review-
(Assignee)

Comment 10

7 years ago
Created attachment 443210 [details] [diff] [review]
patch v3 [Checkin: comment 11]

cf. bug 563419 comment 1.
Attachment #443161 - Attachment is obsolete: true
Attachment #443210 - Flags: review?(neil)

Updated

7 years ago
Attachment #443210 - Flags: review?(neil) → review+
(Assignee)

Comment 11

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

http://hg.mozilla.org/comm-central/rev/61a20d4f19f5
Attachment #443210 - Attachment description: patch v3 → patch v3 [Checkin: comment 11]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
You need to log in before you can comment on or make changes to this bug.