[OS/2] bring seamonkey default UI up-to-date

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Themes
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Walter Meinl, Assigned: Walter Meinl)

Tracking

Trunk
seamonkey2.1b2
x86
OS/2
Dependency tree / graph

Firefox Tracking Flags

(status1.9.1 .13-fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

7 years ago
User-Agent:       Mozilla/5.0 (OS/2; Warp 4.5; rv:2.0b11pre) Gecko/20110201 Firefox/4.0b11pre SeaMonkey/2.1b2pre
Build Identifier: Mozilla/5.0 (OS/2; Warp 4.5; rv:2.0b11pre) Gecko/20110201 Firefox/4.0b11pre SeaMonkey/2.1b2pre

Beside our few additions/substitutions in the toolkit themes we own a single toolbar.css for the suite default theme. The file has gotten out of date after checkins for 3 bugs. All 3 affect the trunk theme, one of them even SeaMonkey-2.0.x.

Reproducible: Always
(Assignee)

Comment 1

7 years ago
Created attachment 508872 [details] [diff] [review]
suite trunk patch [check-in comment 6]
Assignee: nobody → wuno
Status: NEW → ASSIGNED
(Assignee)

Comment 2

7 years ago
Comment on attachment 508872 [details] [diff] [review]
suite trunk patch [check-in comment 6]


>+toolbaritem > menubar,              <- bug585601
>+toolbar > menubar {                 <- bug471372
>+  border-bottom: 0px none;             this affects 2.0.x, too
>+}                                      (a thin extra line between menus and toolbar)
>+
>@@ -90,8 +95,20 @@ toolbargrippy[tbgrippy-collapsed="true"]
>+
>+/* ::::: solid background status bar panel ::::: */ <-bug579731
>+
>+.statusbarpanel-backgroundbox {
>+  -moz-box-align: stretch;
>+  padding: 0px;
>+}
>+
>+.statusbarpanel-backgroundbox > .statusbarpanel-contentbox {
>+  padding: 0px 1px;
>+  -moz-box-align: center;
>+}
Attachment #508872 - Flags: review?(daveryeo)
(Assignee)

Updated

7 years ago
Blocks: 585601, 471372, 579731

Updated

7 years ago
Attachment #508872 - Flags: review?(daveryeo) → review+

Comment 3

7 years ago
Is this ready for check-in?
Version: unspecified → Trunk

Comment 4

7 years ago
(In reply to comment #3)
> Is this ready for check-in?

Please
Keywords: checkin-needed
(Assignee)

Comment 5

7 years ago
Created attachment 509212 [details] [diff] [review]
patch for v 2.0.x [check-in comment 7]

on comm-1.9.1 we missed the follow-up to bug47172 the others were checked-in after branching
Attachment #509212 - Flags: review?(daveryeo)

Comment 6

7 years ago
Comment on attachment 508872 [details] [diff] [review]
suite trunk patch [check-in comment 6]

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/ba87ca371e88
Leaving open for branch patch.
Attachment #508872 - Attachment description: suite trunk patch → suite trunk patch [check-in comment 6]

Updated

7 years ago
Keywords: checkin-needed

Updated

7 years ago
Attachment #509212 - Flags: review?(daveryeo) → review+
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: att 509212 to comm-1.9.1, a=NPOTB OS/2 UI file only

Comment 7

7 years ago
Comment on attachment 509212 [details] [diff] [review]
patch for v 2.0.x [check-in comment 7]

pushed to comm-191
http://hg.mozilla.org/releases/comm-1.9.1/rev/28d6e980893c

Sorry for the delay.
Attachment #509212 - Attachment description: patch for v 2.0.x → patch for v 2.0.x [check-in comment 7]

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
status1.9.1: --- → .13-fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: att 509212 to comm-1.9.1, a=NPOTB OS/2 UI file only
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.