Closed
Bug 726317
Opened 12 years ago
Closed 12 years ago
Remove the side margins in Today Pane
Categories
(Calendar :: General, defect)
Calendar
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.5
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(4 files)
4.38 KB,
patch
|
Fallen
:
review+
andreasn
:
ui-review+
|
Details | Diff | Splinter Review |
33.68 KB,
image/png
|
Details | |
37.91 KB,
image/png
|
Details | |
22.22 KB,
image/png
|
Details |
All elements in the main view like the ltn-sidebar, the unifinder, the calendar-view and also all elements of TB have no margin with a border around them. Only the Today Pane has this.
Assignee | ||
Updated•12 years ago
|
Attachment #596343 -
Attachment is patch: true
Attachment #596343 -
Flags: ui-review?(nisses.mail)
Attachment #596343 -
Flags: review?(philipp)
Assignee | ||
Comment 1•12 years ago
|
||
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Comment 3•12 years ago
|
||
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Assignee | ||
Comment 4•12 years ago
|
||
The today-pane-splitter for Win7 and Linux has a double border because Philipp asked for this a long time ago.
Comment 5•12 years ago
|
||
Comment on attachment 596343 [details] [diff] [review] Remove the margins r=philipp I'm happy to re-evaluate the double-border, I think my concerns were that it would not be obvious that it can be dragged. I'll leave that to Andreas.
Attachment #596343 -
Flags: review?(philipp) → review+
Comment 6•12 years ago
|
||
Tested on Linux and Mac so far and looks good! I don't think you need the double border as I think it don't really make it look more draggable, and the mouse cursor should be enough as it is everywhere else, but it looks fine like this too. Will test on windows as well before I give ui-r+
Comment 7•12 years ago
|
||
Comment on attachment 596343 [details] [diff] [review] Remove the margins And works great in Windows on all themes too.
Attachment #596343 -
Flags: ui-review?(nisses.mail) → ui-review+
Assignee | ||
Comment 8•12 years ago
|
||
Pushed to comm-central: <http://hg.mozilla.org/comm-central/rev/530066c36f5e>
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.5
You need to log in
before you can comment on or make changes to this bug.
Description
•