Closed Bug 398303 Opened 13 years ago Closed 13 years ago

Attendees preview is broken in new event summary dialog

Categories

(Calendar :: Calendar Views, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: omarb.public, Assigned: michael.buettner)

Details

Attachments

(2 files)

Attached image Screenshot
Opening a summary dialog for event with attendees shows all available icons next to attendee:

1) Subscribe to Firefox Team calendar (http://www.google.com/calendar/ical/pdighgf028nmbjbrno8oed8vsg@group.calendar.google.com/public/basic.ics)
2) Mark the calendar as read only
3) Open 'Leek Status Meeting' on 2nd of October

Result:
see attached screenshot
Flags: blocking-calendar0.7?
Tested with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.8pre) Gecko/20071001 Sunbird/0.7 and lightning 2007100203
needs to be investigated for 0.7
Flags: blocking-calendar0.7? → blocking-calendar0.7+
Assignee: nobody → michael.buettner
The summary dialog sets the 'status'-attribute for each attendee icon image to the corresponding participation status of each attendee object, see [1]. This gets picked up by the css rules for the image, see [2]. Admittedly, if the participation status contains any value for which we don't have a matching css rule, we currently just display the whole image. In this particular case the participation status contains "ORGANIZER".

RFC 2445 states that the following participation status values can be defined for a VEVENT:

"NEEDS-ACTION"
"ACCEPTED"
"DECLINED"
"TENTATIVE"
"DELEGATED"

We also don't have an image (and therefore no rule for DELEGATED), which will cause the same problem has any bogus value. A short-term solution could be to don't display an icon image in such cases. Daniel, what's your opinion on that?

[1] http://lxr.mozilla.org/mozilla1.8/source/calendar/base/content/calendar-summary-dialog.js#294
[2] http://lxr.mozilla.org/mozilla1.8/source/calendar/base/themes/winstripe/calendar-event-dialog.css#54
Attached patch patch v1Splinter Review
This patch suppresses the display of the attendee icon image for unknown participation status values, i.e. those we currently don't have an image for.
Attachment #283537 - Flags: review?(daniel.boelzle)
(In reply to comment #3)
...
> We also don't have an image (and therefore no rule for DELEGATED), which will
> cause the same problem has any bogus value. A short-term solution could be to
> don't display an icon image in such cases. Daniel, what's your opinion on that?
I'm fine with that. Christian/Stella may propose a different icon in the future, e.g. a question mark.
Comment on attachment 283537 [details] [diff] [review]
patch v1

r=dbo
Attachment #283537 - Flags: review?(daniel.boelzle) → review+
patch checked in on trunk, MOZILLA_1_8_BRANCH and SUNBIRD_0_7_BRANCH

-> FIXED
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → 0.7
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
verified with
Mozilla/5.0 (Windows; U; Windows NT 5.1; pl; rv:1.8.1.8pre) Gecko/20071002 Sunbird/0.7
Status: RESOLVED → VERIFIED
Damian, how can you verify this bug with that build (and the same with Bug 375390)? It was reported with 20071002 build
Verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.8pre) Gecko/20071006 Sunbird/0.7
and
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a9pre) Gecko/2007100601 Calendar/0.6a1
You need to log in before you can comment on or make changes to this bug.