> On July 15, 2014, 2:04 a.m., Sebastian Kügler wrote: > > src/declarativeimports/calendar/qml/DaysCalendar.qml, line 58 > > <https://git.reviewboard.kde.org/r/119283/diff/2/?file=290254#file290254line58> > > > > + root.borderWidth > > > > The last cell has two borders here. Can you verify? Same goes for the > > height.
grmblz, htmlification outsmarted me: has to be " ... + root.borderWidth". - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119283/#review62365 ----------------------------------------------------------- On July 14, 2014, 9:46 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119283/ > ----------------------------------------------------------- > > (Updated July 14, 2014, 9:46 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > This patch simplifies the grid a bit more by using Canvas instead of > Rectangles, so actually lowering the number of Items needed to create the > grid from 16 to just 1. > > It works pretty neatly with no junctions problem at all. > > > Diffs > ----- > > src/declarativeimports/calendar/qml/DayDelegate.qml d626441 > src/declarativeimports/calendar/qml/DaysCalendar.qml 1249de2 > src/declarativeimports/calendar/qml/MonthView.qml 4946635 > > Diff: https://git.reviewboard.kde.org/r/119283/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > Screenshot before/after > > https://git.reviewboard.kde.org/media/uploaded/files/2014/07/14/e744e403-89ff-4947-95eb-3ade4823ebb5__plasma_cal_canvas.png > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
