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

David Edmundson david at davidedmundson.co.uk
Thu Oct 29 17:06:33 UTC 2015


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



applets/digital-clock/package/contents/ui/CalendarView.qml (line 158)
<https://git.reviewboard.kde.org/r/125864/#comment60163>

    FYI you can leave it as anchors.margins: spacing
    
    it means you're setting that anchorMargin you're not using, but given you don't have a left anchor it won't do anything anyway.


- David Edmundson


On Oct. 29, 2015, 4: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, 4: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/bb2e232a/attachment.html>


More information about the Plasma-devel mailing list