Review Request: Re-enable showing all holiday types in plasma calendar

[John Layt]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4236/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

The calendar data engine will now return all holidays along with their holiday type, i.e. if a day off or not.  The calendar plasmoid only shows days off highlighted in red.  In the pop-up only days off are prefixed with 'Holiday', but all other holidays are now listed.

I have also changed how Events are displayed.  They were shown as a green highlight with higher priority than a holiday.  This caused two issues.  First, information is blocked, it can only show a day is a Holiday or an Event, it can't show when you have both on the one day.  Second, many users will have Events on almost every day, so almost every day would be green highlighted, which besides looking ugly and busy also effectively wastes a high visibility signal on a more common piece of information.  Instead I've gone for the more standard bold day number as done in KOrganizer and most other calendar programs, and re-used the green highlight for Holidays that are not days off.

In the future we could use other options such as cell shading and font colour, and make it user configurable.

Screenies attached.


This addresses bug 218549.
    https://bugs.kde.org/show_bug.cgi?id=218549


Diffs
-----

  trunk/KDE/kdebase/workspace/libs/plasmaclock/calendartable.h 1134154 
  trunk/KDE/kdebase/workspace/libs/plasmaclock/calendartable.cpp 1134154 
  trunk/KDE/kdebase/workspace/plasma/generic/dataengines/calendar/calendarengine.h 1133276 
  trunk/KDE/kdebase/workspace/plasma/generic/dataengines/calendar/calendarengine.cpp 1133276 

Diff: http://reviewboard.kde.org/r/4236/diff


Testing
-------

Always :-)  Note in the second screenie we have Feb 12 with a public holiday (day off), a commemorative holiday (not a day off), and an event.


Screenshots
-----------

Calendar Table
  http://reviewboard.kde.org/r/4236/s/418/
Calendar Popup
  http://reviewboard.kde.org/r/4236/s/420/


Thanks,

John