----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
