366680 - Event tool tips in calendar views should display list attendees

Status: ASSIGNED

(Calendar :: Calendar Frontend, enhancement)

Description

[Christian Jansen]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.8.1.1) Gecko/20061204 Firefox/2.0.0.1
Build Identifier: 

Attendees assigned to an event should be listed in the tool tip which occur during mouse over. Currently users need at least double click the event to get this information. Beneath to the invitees name, the participation status should also be displayed.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Some questions have come up while creating a patch:

* Should the participation status be text or icons?
* Should only the first few participants be shown?
  - If so, how many?
* If we support the DELEGATED state, do we also want to show DELEGATED-FROM?
* Should any further information be shown?

Comment 2

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Screenshot with tooltip

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Details from the QA Chat:

* Participation Status should be an icon (before the name)
* DELEGATED-FROM is too far out on the horizon to care about for now
* The size of the tooltip should be relative to the size of the application window
* WCAP uses a commonName. New format will be:
  (icon) commonName <[e-mail]>
* The icons need to be VERY descriptive since there should be no tooltip in tooltip. The prototype icons are a good start, but we need more.

Christian, if you have any additional comments I'd be happy to implement this :)

Comment 4

[Christian Jansen]

(In reply to comment #3)
> Details from the QA Chat:
> 
> * Participation Status should be an icon (before the name)
Agreed
> * DELEGATED-FROM is too far out on the horizon to care about for now

Agreed
> * The size of the tooltip should be relative to the size of the application
> window

> * The icons need to be VERY descriptive since there should be no tooltip in
> tooltip. The prototype icons are a good start, but we need more.

Which one? We have "I will attend", "tentative" and "I will not attend" 
> 
> Christian, if you have any additional comments I'd be happy to implement this
> :)
> 

I think it would make sense to display:
Title, Location, Date, Organizer, (as Link (mailto:)), all invites, Description & if available the URL as hyper link.

> * WCAP uses a commonName. New format will be:
>   (icon) commonName <[e-mail]>

do we really need to display both? I don't think so. If the real name is available good, otherwise we display the e-mail address only. This would reduce redundant information. If users want to get more details they can open the event dialog. Displaying the name or e-mail address only would allow to display a larger number of invitees, because we could do 2 rows.

The event summary dialog is described here:
http://wiki.mozilla.org/Calendar:SMB_Event_Summary


I'll attach a mock-up which shows the stuff more visually :-)

Comment 5

[Christian Jansen]

Attachment: Tooltip Mock-Up displaying the stuff decribed in comment #4

Comment 6

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

> * The size of the tooltip should be relative to the size of the application window

What about this? Of course if there is not enough information, the tooltip should not be larger, but if we have a lot of attendees, we could make the tooltip larger if the app window is also large. This way the tooltip will never be larger than the app and if its fullscreen we can still see a lot of information.

> Which one? We have "I will attend", "tentative" and "I will not attend" 
What about "NEEDS-ACTION", "DELEGATED", "COMPLETED", "IN PROGRESS" ? I know these are fields that our app doesn't really set, but what if you get a calendar from a different source that has these values set? The minimal thing we might need is an icon maybe with a question mark that show up on all those statii.

> I think it would make sense to display:
> Title, Location, Date, Organizer, (as Link (mailto:)), all invites, Description
> & if available the URL as hyper link.
> 
> > * WCAP uses a commonName. New format will be:
> >   (icon) commonName <[e-mail]>
> 
> do we really need to display both? I don't think so. 
If the organizer should be a link, why not make the attendees links too? If there is a commonName and an email, show the commonName and link to the email. If there is no commonName but an email, show the email and link to it.

Comment 7

[Christian Jansen]

>> * The size of the tooltip should be relative to the size of the application window
> 
> What about this? Of course if there is not enough information, the tooltip
> should not be larger, but if we have a lot of attendees, we could make the
> tooltip larger if the app window is also large. This way the tooltip will never
> be larger than the app and if its fullscreen we can still see a lot of
> information.
this would be ok for me, I never seen a tool poster before :-) 
> 
>> Which one? We have "I will attend", "tentative" and "I will not attend" 
> What about "NEEDS-ACTION", "DELEGATED", "COMPLETED", "IN PROGRESS" ? I know
> these are fields that our app doesn't really set, but what if you get a
> calendar from a different source that has these values set? The minimal thing
> we might need is an icon maybe with a question mark that show up on all those
> statii.

Aren't "NEEDS-ACTION", "DELEGATED", "COMPLETED", "IN PROGRESS" task values? We should display the PARTSTAT parameters only. A question mark would be understood as tentetive. Maybe it makes most sense to display no icon (just an idea...)

> 
>> I think it would make sense to display:
>> Title, Location, Date, Organizer, (as Link (mailto:)), all invites, Description
>> & if available the URL as hyper link.
>> 
>> > * WCAP uses a commonName. New format will be:
>> >   (icon) commonName <[e-mail]>
>> 
>> do we really need to display both? I don't think so. 
> If the organizer should be a link, why not make the attendees links too? 

Makes sense. If we change that we need to change this in the Events summary dialog, too. Mickey, what do you think?
If
> there is a commonName and an email, show the commonName and link to the email.
ok
> If there is no commonName but an email, show the email and link to it.
ok


What are we going to display for tasks? I think for the first step "Title", "Category", "Start", "Due Date:", "Status" , "Completeness", "Description" & "URL" would be good.

Comment 8

[Daniel Boelzle [:dbo]]

(In reply to comment #7)
> Aren't "NEEDS-ACTION", "DELEGATED", "COMPLETED", "IN PROGRESS" task values? We

Not quite, <http://tools.ietf.org/html/rfc2445#section-4.2.12> specifies the possible PARTSTAT of attendees in events and todos. NEEDS-ACTION is available for both events and todos while COMPLETED and IN-PROGRESS only for todos. 

Comment 9

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

After a complete rewrite of the tooltip stuff, I'm almost there. I also have some nice ideas for an extension that allows you to totally customize what is shown in the tooltip.

A thing that occurred to me: If we use links for organizers and attendees, they will just sit there and look nice. The user will have no possibility to click on the link, since the tooltip goes away when you move away from the element. Should we still make them look like links?

I'm probably not quite understanding the RFC. From what you guys are saying and the RFC, I assume an ATTENDEE can have "ACCEPTED", "DECLINED", "TENTATIVE", "NEEDS-ACTION" and "DELEGATED". 

I see 4 icons in sun-calendar-event-dialog-attendees.png. The css file is a bit confusing though, it seems as if the white icon is used for tentative and the yellow one is for needs-action. I would have assumed its the other way around.

Where should the icons go? Right now for testing I have included the file in /m/c/prototypes/... in /m/c/base/jar.mn, but we probably want the icon files in /m/c/base/themes/common/ right?

Comment 10

[Michael Büttner (no reviews TFN)]

(In reply to comment #9)
> I'm probably not quite understanding the RFC. From what you guys are saying
> and the RFC, I assume an ATTENDEE can have "ACCEPTED", "DECLINED",
> "TENTATIVE", "NEEDS-ACTION" and "DELEGATED". 
I assume you understand the RFC correctly but the invitation related stuff in the event dialog is not quite final. I'm currently ignoring "TENTATIVE" and "DELEGATED", just using the states that I'm having icons for :-) This needs to be finalized, for sure.

> Where should the icons go? Right now for testing I have included the file in
> /m/c/prototypes/... in /m/c/base/jar.mn, but we probably want the icon
> files in/m/c/base/themes/common/ right?
Right, we want all those icons to live in m/c/base/themes/common. But for now, you can refer to the files in /m/c/prototypes and I'll change that when moving the prototype dialog to its final location.

Comment 12

[Simon Nussbaum]

(In reply to comment #11)
> *** Bug 402023 has been marked as a duplicate of this bug. ***
> 

What's the status of this bug? I'm asking because you wrote that you are almost there. I like the idea of an extension that provides full customization. 

Unfortunately I stumbled over this bug report too late. I already added some lines to the mouseoverPreviews.js-File to provide Attendees in the tool tips.

Comment 14

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: WiP Patch - v2

Simon, sorry I've left this bug behind for a while. There are some aggravating issues with branch and this bug that keep me from fixing this bug in a good-looking manner.

This is my WiP patch, which is really old. Maybe someone wants to take over, if they find time before I do. Note I haven't touched this patch in almost a year, so don't expect anything to work.

Comment 15

[Guilherme Gaspar]

Simon, can you tell us or send an attach to we know how to resolv this issue ? I am using lightning 0.8 and want to see attendees status too.

Thanks 

Comment 16

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

This issue is far from resolved, due to some issues on branch. You'll have to do some development to make it work. Interested?

Comment 17

[Guilherme Gaspar]

Hello Philipp. We are here to help, or at least try :) I see the other bugs related and the problem is with the size of tooltip right?

Comment 18

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: De bitrotted patch - v3

Guilherme, in case you are still interested, I have de-bitrotted this patch allowing it to apply to the current tree.

This patch probably needs some style cleanup and general code cleanup, testing on windows and mac, and possibly letting the tooltip stay until the mouse leaves it.

Strangely enough, on my linux machine the sizeTo() hacks are not needed. This may be different on windows though.

Comment 19

[Graham Perrin]

Reading this alongside bug 543744 comment 17 and thereabouts.

Comment 20

[arjun mehta]

Hope this can be implemented soon.

Perhaps the invitee list could be truncated at 10 invitees so the tooltip doesn't get too large, but the mockup that Philipp Kewisch added in comment 2 would be functional enough, though if icons really wanted to be used, there is an icon system when you double click into the event details.

Either way, attendees ought to be surfaced in an easier way for events if Lightning is to cater to enterprise level schedules. (Meetings when knowing the specifics of attendees is extremely important).