489773 - Invitation dialog has flaws

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Daniel Boelzle [:dbo]]

Get multiple invitations, then open the invitation dialog (press the link) in the footer bar: Selecting an invitation shows the accept/decline buttons, but the previous selection does not hide its buttons nor is the new selection hilighted.

Comment 1

[Martin Schröder [:mschroeder]]

Attachment: Patch v1

* Selection was missing because the used background image has been removed from toolkit. Replaced it with standard highlight color.
* Corrected color values for listbox items for better visibility.
* Replaced event handler for button handling with CSS solution.
* Corrected -moz-user-focus, so you can just tab the buttons and the selected listbox item.

Comment 2

[Philipp Preißer-Kewisch [:Fallen][☁️📆]]

Comment on attachment 381367 [details] [diff] [review]
Patch v1

> calendar-invitations-richlistbox {
>   -moz-binding: url(chrome://calendar/content/calendar-invitations-list.xml#calendar-invitations-richlistbox);
>-  -moz-user-focus: normal;
>+  -moz-user-focus: ignore;
> }
I think the default is -moz-user-focus: ignore. Please check this and remove the style accordingly.

> calendar-invitations-richlistitem[selected="true"] {
>-  background-image: url("chrome://mozapps/skin/downloads/downloadSelected.png") !important;

Try this instead:
background-image: url(chrome://mozapps/skin/extensions/itemEnabledFader.png);

Comment 3

[Martin Schröder [:mschroeder]]

(In reply to comment #2)
> I think the default is -moz-user-focus: ignore. Please check this and remove
> the style accordingly.
Yep... removed twice!

> Try this instead:
> background-image: url(chrome://mozapps/skin/extensions/itemEnabledFader.png);
Works! :)

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/67ab9ddecfbe>

-> FIXED