Review Request 124072: Refactor the Calendar grid computation a bit

[Mark Gaiser]

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



src/declarativeimports/calendar/qml/DaysCalendar.qml (lines 31 - 32)
<https://git.reviewboard.kde.org/r/124072/#comment55757>

    Can you fix the double root.borderWidth addition? It looks odd.
    
    If not, then please add more parenthesis.


+1

- Mark Gaiser


On jun 11, 2015, 2:42 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124072/
> -----------------------------------------------------------
> 
> (Updated jun 11, 2015, 2:42 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Makes the code a bit simpler & lighter and fixes the sometimes missing bottom line in calendar.
> 
> The grid is now equally padded from both sides (including the month name), so basically it's now
> always aligned to the center and this also fixes the cases where there was a bigger padding on
> one side than on the other.
> 
> The grid is now also anchored to the bottom so that the bottom margin can stay moreless consistent
> with the side margins in different sizes.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/calendar/qml/MonthView.qml 86e37d8 
>   src/declarativeimports/calendar/qml/DaysCalendar.qml 5308708 
> 
> Diff: https://git.reviewboard.kde.org/r/124072/diff/
> 
> 
> Testing
> -------
> 
> Lots of 600x-zoom inspections and pixel counting. Tested both in Calendar applet and Digital Clock applet.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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