Closed
Bug 298360
Opened 20 years ago
Closed 18 years ago
calendar list needs better visible/not display
Categories
(Calendar :: Lightning Only, defect)
Calendar
Lightning Only
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 368558
Lightning 0.1
People
(Reporter: shaver, Assigned: jminta)
Details
Attachments
(4 files)
|
6.98 KB,
patch
|
dmosedale
:
first-review+
|
Details | Diff | Splinter Review |
|
5.07 KB,
patch
|
Details | Diff | Splinter Review | |
|
5.00 KB,
image/png
|
Details | |
|
38.49 KB,
image/png
|
Details |
Should try checkbox image, and custom white-plus-transparent overlays for calendar-color background. (WTB: <treecol type="checkbox">)
| Reporter | ||
Updated•20 years ago
|
Blocks: lightning-0.1
Updated•20 years ago
|
Target Milestone: --- → Lightning 0.8
| Assignee | ||
Comment 1•19 years ago
|
||
Replaces the current red background with checkbox images. Basically a straight port of what Sunbird does for its tasklist. Doesn't do anything with calendar colors yet, that'll take something a bit fancier.
Attachment #197481 -
Flags: first-review?(dmose)
Comment 2•19 years ago
|
||
Comment on attachment 197481 [details] [diff] [review] add checkbox images r=dmose
Attachment #197481 -
Flags: first-review?(dmose) → first-review+
Updated•19 years ago
|
Assignee: shaver → jminta
| Assignee | ||
Comment 3•19 years ago
|
||
Comment on attachment 197481 [details] [diff] [review] add checkbox images patch checked in. leaving bug open to figure out the calendar colors issue.
| Assignee | ||
Comment 4•19 years ago
|
||
This patch makes the Lightning calendar box look much like Sunbird's by adding an additional column for the calendar color. Alternatively, it's possible to just color the background of the calendar's name cell. I'm not sure if that's a better option from a user-experience standpoint.
Attachment #204051 -
Flags: first-review?(dmose)
Comment 5•19 years ago
|
||
I talked to beltzner a bit about it in IRC, and here are his thoughts: <emose> beltzner: got any opinion on bug 298360? <beltzner> assuming that you can't actually colour the checkboxes, iCal style, it seems like a decent compromise <beltzner> colouring the cell containing the calendar name seems both heavy and like it could lead to contrast problems with reading the name of the calendar <emose> well, we have code to "get contrasting colors" <emose> which is what we use to write the text on the event boxes <emose> which are in the color <emose> but you'd suggest coloring the checkboxes as the best option? <beltzner> if at all possible, yeah <emose> beltzner: ok, thanks <beltzner> if you end up using the multiple columns, I'd suggest making the colour box appear at the RHS, slightly shorter than full cell height and valign=center'd <beltzner> just as visual niceties ;)
Comment 6•19 years ago
|
||
Comment on attachment 204051 [details] [diff] [review] add color column Removing r?, since it sounds like a new patch is forthcoming.
Attachment #204051 -
Flags: first-review?(dmose)
Comment 7•19 years ago
|
||
My two cents: Sunbird currently displays the fields in this order: [checkbox][color][calendarname]. In my opinion it should be the same order in Sunbird and Lightning. So either use the patch as is or use beltzners suggestions for both. Nit: I think the proposed color column is too wide (25px), consuming too much space. My proposal would be to make it square sized or use the same width as the checkbox (18px).
| Assignee | ||
Comment 8•19 years ago
|
||
Here are several different visual presentation options. Option 5 represents my attempt to make the checkboxes more transparent. Not being very well skilled with images, it's not great. In an ideal world we could get Mark Carson to tidy up those checkboxes. beltzner, which would you recommend?
Updated•19 years ago
|
No longer blocks: lightning-0.1
Updated•19 years ago
|
QA Contact: shaver → lightning
Comment 9•19 years ago
|
||
Comment on attachment 204237 [details]
several options
I'll vote for Option 1 (as in Sunbird) or Option 3 (checkbox with colored background).
Updated•18 years ago
|
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•