Make the today-pane sidebar header look like the main toolbar.

RESOLVED FIXED in 6.2

Status

enhancement
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified

Details

Attachments

(4 attachments)

Assignee

Description

Last year
The actual today-pane sidebar header looks different in color and size to the main toolbar.

Make it look like the toolbar, also make it as tall as the toolbar would make it more homogen and makes the pane more a part of the main content.
Assignee

Comment 1

Last year
I moved more code to the common file. On Mac I removed the special spin buttons and use the same as Linux and Windows use. This looks also better on the now taller sidebar header.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8948173 - Flags: review?(philipp)
Comment on attachment 8948173 [details] [diff] [review]
today-pane.patch

Review of attachment 8948173 [details] [diff] [review]:
-----------------------------------------------------------------

Code looks good to me, can you show me a screenshot?
Attachment #8948173 - Flags: review?(philipp) → review+
Assignee

Comment 3

Last year
Posted image Screenshot on Linux
Assignee

Comment 4

Last year
Posted image Screenshot on Mac
Assignee

Comment 5

Last year
Assignee

Updated

Last year
Keywords: checkin-needed
Perfect, looks great! Thanks :)

Comment 7

Last year
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/8ebb03cbea92
Use the toolbar look for the today-pane sidebar header. r=philipp
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

Last year
Target Milestone: --- → 6.2
I think, the two macOS-specific images (spin-buttons{-active}.png) could have been removed with this patch.
Assignee

Comment 9

Last year
In earlier bugs, Philipp wanted to leave no more used files in the tree. If this is no more desired I can make a bug to remove all this files.
Philipp, can you shed some light on the issue discussed in comment#8 and comment#9, please?
Flags: needinfo?(philipp)
Answered in bug 1446748#c57 .
Flags: needinfo?(philipp)
You need to log in before you can comment on or make changes to this bug.