Review Request 126960: [Calendar] Add proper back/forward buttons and a "Today" button

Kai Uwe Broulik kde at privat.broulik.de
Thu Feb 4 23:32:29 UTC 2016



> On Feb. 4, 2016, 11:24 nachm., Heiko Tietze wrote:
> > File Attachment: How about this? #2 - How about this? #2
> > <https://git.reviewboard.kde.org/r/126960/#fcomment500>
> >
> >     Bug 299174 - Remove "week number" from system tray calendar for improved readability 
> >     
> >     https://bugs.kde.org/show_bug.cgi?id=299174
> >     
> >     (I disagree with the idea to remove it. But some ideas have been stressed in this thread.)

You're aware that week numbers in the calendar are optional and turned off by default and were only re-introduced after frequent user complaints.


- Kai Uwe


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


On Feb. 4, 2016, 11:19 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126960/
> -----------------------------------------------------------
> 
> (Updated Feb. 4, 2016, 11:19 nachm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Bugs: 336124, 348362 and 358536
>     http://bugs.kde.org/show_bug.cgi?id=336124
>     http://bugs.kde.org/show_bug.cgi?id=348362
>     http://bugs.kde.org/show_bug.cgi?id=358536
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This removes the custom label-based triangles and replaces them with proper ToolButtons using proper icons. It also adds a "Today" button to return to the current day. Also, tooltips that reflect the actual action ("Previous Month", "Previous Year", "Previous Decade", depending on the zoom level) were added.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml 3ab16eb 
>   src/declarativeimports/calendar/qml/MonthView.qml c876e3b 
> 
> Diff: https://git.reviewboard.kde.org/r/126960/diff/
> 
> 
> Testing
> -------
> 
> Works.
> 
> The weekday names look a bit awkward now
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/02/01/a065dfcf-ca75-4d50-81aa-4d725245344e__Screenshot_20160201_234605.png
> How about this?
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/02/04/d80b6161-3da3-4669-ba7c-19f62edbf542__Screenshot_20160205_001739.png
> How about this? #2
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/02/04/73f2ada9-48cd-4b22-8ef2-5d37f2238442__Screenshot_20160205_001754.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160204/cf35e30c/attachment.html>


More information about the Plasma-devel mailing list