Show tooltips in summary dialog

VERIFIED FIXED in 0.9

Status

VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

unspecified

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 330733 [details] [diff] [review]
Fix - v1

The current attendee/organizer fields show only the name if it is specified by the server. This patch adds a tooltip with name and email and also fixes the focus rectangle to not extend over the whole row.
Attachment #330733 - Flags: ui-review?(christian.jansen)
Attachment #330733 - Flags: review?(Berend.Cornelius)

Comment 1

10 years ago
Comment on attachment 330733 [details] [diff] [review]
Fix - v1

Patch works fine. 
I noticed already that there is a lot of common logic of the task view and the calendar-summary-dialog and therefor I decided to apply all my summary dialog fixes also to the task-view (see Bug 396819 – Event Summary Dialog doesn't show important information to user). I think this should be done in this case too. As you told me this patch is a by-product of another patch and therefor I think it is fine for the time being.

>+    if (organizer && organizer.id) {
>+        var email = organizer.id;
>+        var re = new RegExp("^mailto:(.*)", "i");
>+        var matches = re.exec(email);
>+        if (matches) {
>+            email = matches[1];
>+        }
We have this logic also in many places in our project. I think we should some day think about consolidating this, too.
Attachment #330733 - Flags: review?(Berend.Cornelius) → review+

Updated

10 years ago
Attachment #330733 - Flags: ui-review?(christian.jansen) → ui-review+

Comment 2

10 years ago
ui=christian
(Assignee)

Comment 3

10 years ago
Created attachment 331475 [details] [diff] [review]
Fix - v2

calAttendee has a toString() method that takes care of composing the email string, I've used that. I also tried to create a binding that works for both the organizer label and the listitem labels, but that failed due to other XUL issues. There is a XUL issue with this patch (and v1) too, but see the comments in the patch.
Attachment #330733 - Attachment is obsolete: true
Attachment #331475 - Flags: ui-review+
Attachment #331475 - Flags: review+
(Assignee)

Comment 4

10 years ago
Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9

Comment 5

10 years ago
Checked in lightning 2008072904 -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.