Closed
Bug 361211
Opened 18 years ago
Closed 18 years ago
single selected event doesn`t change color when deselected
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: Taraman, Assigned: jminta)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mattwillis
:
first-review+
mvl
:
second-review+
mvl
:
ui-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Lightning 0.4a1 (build 1006111906) in Thunderbird 1.5.0.8 (20061025)
When I select an item an then click on empty space in a day (not selecting another item), the item keeps the "selected color".
Occurs in all views (month, week, day).
Reproducible: Always
Steps to Reproduce:
1. click on an event
2. click on empty space in calendar
Actual Results:
color stays "selected"
Expected Results:
it should change back to the color for this calendar
Confirmed on Sunbird
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20061201 Calendar/0.4a1
Build ID = 2006120104
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•18 years ago
|
||
First, we stop clicks on items from bubbling up. Then, any clicks that do hit our containers we know missed items, so we should unselect everything.
(It'd be nice if our containers inherited from a common binding, but that's just a dream of mine.)
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #250783 -
Flags: ui-review?(mvl)
Attachment #250783 -
Flags: second-review?(mvl)
Attachment #250783 -
Flags: first-review?(lilmatt)
Comment 3•18 years ago
|
||
Comment on attachment 250783 [details] [diff] [review]
unselect when clicking elsewhere
r=lilmatt
Looks fine to me.
Attachment #250783 -
Flags: first-review?(lilmatt) → first-review+
Reporter | ||
Comment 4•18 years ago
|
||
Works for me as well.
THX
Comment 5•18 years ago
|
||
Comment on attachment 250783 [details] [diff] [review]
unselect when clicking elsewhere
Most other apps also unselect on clicks. I hope it won't be too annoying by deselecting when you don't want to, but it never annoyed me in other apps. So i have some confidence in the model.
ui-review+r2=mvl
Attachment #250783 -
Flags: ui-review?(mvl)
Attachment #250783 -
Flags: ui-review+
Attachment #250783 -
Flags: second-review?(mvl)
Attachment #250783 -
Flags: second-review+
Assignee | ||
Comment 6•18 years ago
|
||
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment 7•18 years ago
|
||
verified with
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a2pre) Gecko/20070112 Calendar/0.4a1
Status: RESOLVED → VERIFIED
Whiteboard: [litmus testcase wanted]
You need to log in
before you can comment on or make changes to this bug.
Description
•