Closed Bug 298360 Opened 20 years ago Closed 18 years ago

calendar list needs better visible/not display

Categories

(Calendar :: Lightning Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 368558
Lightning 0.1

People

(Reporter: shaver, Assigned: jminta)

Details

Attachments

(4 files)

Should try checkbox image, and custom white-plus-transparent overlays for
calendar-color background.

(WTB: <treecol type="checkbox">)
Target Milestone: --- → Lightning 0.8
Attached patch add checkbox images — — Splinter Review
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 on attachment 197481 [details] [diff] [review]
add checkbox images

r=dmose
Attachment #197481 - Flags: first-review?(dmose) → first-review+
Assignee: shaver → jminta
Comment on attachment 197481 [details] [diff] [review]
add checkbox images

patch checked in. leaving bug open to figure out the calendar colors issue.
Attached patch add color column — — Splinter Review
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)
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 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)
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).
Attached image several options —
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?
No longer blocks: lightning-0.1
QA Contact: shaver → lightning
Comment on attachment 204237 [details]
several options

I'll vote for Option 1 (as in Sunbird) or Option 3 (checkbox with colored background).
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.

Attachment

General

Creator:
Created:
Updated:
Size: