----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125891/#review87784 -----------------------------------------------------------
Thanks for adding usability! I agree with Kai that the whole header area should be scrollable instead of just the label. One thing to make sure (if that is even affected by this change at all) is that when we have events in the poup, scrolling over the even description won't change the month. Other than that, +1 for the change! - Thomas Pfeiffer On Oct. 31, 2015, 4:21 p.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125891/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2015, 4:21 p.m.) > > > Review request for Plasma and KDE Usability. > > > Repository: plasma-framework > > > Description > ------- > > Allow to scroll navigate back/forward in DaysCalendar in the entire > DaysCalendar component, not just on header label. > > > Diffs > ----- > > src/declarativeimports/calendar/qml/DaysCalendar.qml a1d84d4 > > Diff: https://git.reviewboard.kde.org/r/125891/diff/ > > > Testing > ------- > > > Thanks, > > David Rosca > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
