[Plone-UI] Review of plone.app.event (plip 10886)

[Roché Compaan]

plone.app.event has already been merged with 4.4 but it needs a lot of
UI love before one can call it published.

Here's my feedback on plip 10886 below for tonight's meeting. Should I
post my review on the plip itself? And log tickets on the
plone.app.event repo?

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 one would expect a user to 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
repeats 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 I keep that on the
heading style.

Event view:

12. The view of occurences in the "View" of the event needs a lot of
love. The first view events is shown as a bulleted list and then the
gap
is indicated with 3 dots below each other and then 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.


--
Roché Compaan
Upfront Systems                   http://www.upfrontsystems.co.za