Review Request 125864: Add back the Agenda part of the Calendar

[Martin Klapetek]

> On Oct. 29, 2015, 5:55 p.m., Sebastian Kügler wrote:
> > applets/digital-clock/package/contents/ui/CalendarView.qml, line 99
> > <https://git.reviewboard.kde.org/r/125864/diff/1/?file=413259#file413259line99>
> >
> >     if (updatedData === monthView.currentDate) ?
> >     
> >     I don't know if js date comparisons work that well, but seems simpler and clearer...

No, this wouldn't work, there was a lengthy discussion yesterday in #plasma about this. A summary:

<ChALkeR> But that compares references.
<ChALkeR> {} == {} ? false

...javascript.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125864/#review87678
-----------------------------------------------------------


On Oct. 29, 2015, 5:24 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125864/
> -----------------------------------------------------------
> 
> (Updated Oct. 29, 2015, 5:24 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Adds back the Agenda that was there pre-5.0 with some changes.
> 
> It now shows an overview of the given day events/holidays. I'd
> like to add a second view at some point that would show you the
> selected day by hours but that is much more work and the freeze
> is getting closer, so maybe that will have to wait till 5.6.
> 
> 
> Diffs
> -----
> 
>   applets/digital-clock/package/contents/ui/CalendarView.qml b52eb10 
> 
> Diff: https://git.reviewboard.kde.org/r/125864/diff/
> 
> 
> Testing
> -------
> 
> Selecting a day correctly changes the date in the agenda and correctly displays the events of the day.
> 
> Event removal/modification also removes/modifies it from the agenda.
> 
> 
> File Attachments
> ----------------
> 
> Current look in Breeze
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/29/5485250c-8d20-409e-8a5b-0eda7a9d7b8d__snapshot29.png
> Multiple same-day events
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/29/d453a01e-f9cf-4da3-8f6a-fac5aa6aadf5__snapshot18.png
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151029/8f7d3171/attachment-0001.html>