Attendees preview is broken in new event summary dialog

VERIFIED FIXED in 0.7

Status

defect
VERIFIED FIXED
12 years ago
12 years ago

People

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

Tracking

unspecified
Bug Flags:
blocking-calendar0.7 +

Details

Attachments

(2 attachments)

Reporter

Description

12 years ago
Posted 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?
Reporter

Comment 1

12 years ago
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

Updated

12 years ago
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
Posted 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
Assignee

Updated

12 years ago
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED

Comment 8

12 years ago
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
Reporter

Comment 9

12 years ago
Damian, how can you verify this bug with that build (and the same with Bug 375390)? It was reported with 20071002 build
Reporter

Comment 10

12 years ago
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.