[PLIP-Advisories] [Plone development workspace] #10886: plone.app.event - new eventtype for plone

Change notifications for Plone PLIPs on Trac. plone-plip-advisories at lists.plone.org
Wed Mar 13 07:15:31 UTC 2013


#10886: plone.app.event - new eventtype for plone
----------------------+------------------------------------
  Reporter:  thet     |      Owner:  thet
      Type:  PLIP     |     Status:  reopened
  Priority:  major    |  Milestone:  4.4
 Component:  General  |    Version:  4.1
Resolution:           |   Keywords:  event, recurring, date
----------------------+------------------------------------
Changes (by rochecompaan):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 On behalf of the UI team I did a review of plone.app.event on the latest
 coredev buildout, ie on the 4.4 branch. Awesome work, the functionality
 looks great! The UI needs attention in quite a few places though. I don't
 have solutions for all the issues, we are still exploring UI solutions to
 some of the problems in the UI team. Some issue below are low hanging
 fruit and should be easy to fix though.

 1. When installing a new Plone site without selecting any extensions,
 there is no event type. I would expect the plone.app.event dexterity
 extension to be installed by default.

 2. Not convinced we should have the calender portlet installed by default.

 3. Bold styling for month days in Calendar portlet is too overbearing.

 Editing an event:

 4. One can specify start and end time even if "Whole day" is checked.

 5. The description for the recurrence widget reads "RFC5545 compatible
 recurrence definition". I guess this must still be fixed.

 6. The checkbox combined with "Does not repeat" is confusing UI. One would
 expect that ticking a checkbox is a confirmation of what is shown on the
 label, so a user will click on "Does not repeat" if the user really does
 not want the event to repeat. Instead, what happens now, is that the label
 toggles and the event now repeats with 10 occurrences by default.

 7. I'm not convinced we should have 10 daily repeats for an event by
 default. It shouldn't really be necessary to show 10 daily repeats just to
 show the user that the event can repeat, the UI for creating repeating
 events should be obvious enough. I'll try and think of better solution.

 8. Batch navigation is shown "[1 - 10]", even if the batch is smaller than
 10.

 9. The date format (dd/mm/yy) for the "Until" input in the recurrence
 widget is not really human readable. Rather format with full month name,
 including the week day, eg Saturday January 05, 2013.

 10. If I edit the start date of the event, the recurrence widget doesn't
 update the start of the occurrence.

 11. The heading style for "Repeat", "Selected Dates" and "Add" in the
 recurrence overlay is inconsistent with the heading styles in public.css.
 #888888 is too subdued, just use plain black. I think the bottom border
 helps to separate the sections so keep that on the heading style.

 Event view:

 12. The view of occurences in the "View" of the event needs some love. The
 first few events is shown as a bulleted list and then the gap is indicated
 with 3 dots below each other and then suddenly the final event is shown.
 There doesn't seem to be a way for a user with only view permission to see
 all occurrences on an event. I don't think actual occurrences should be
 shown in the "eventDetails" view, describing the occurrence would be much
 more helpful, eg "This event repeats weekly on a Wednesday for the next 10
 weeks." Additionally one can perhaps link to "Show all occurrences" which
 pops up and overlay showing a view with all occurences.

-- 
Ticket URL: <http://dev.plone.org/ticket/10886#comment:60>
Plone development workspace <https://dev.plone.org/>
Plone Enterprise Content Management System


More information about the PLIP-Advisories mailing list