----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119916/#review66247 -----------------------------------------------------------
Ship it! And then we need to fix the horrible Free Space Notifier notification ;) - Kai Uwe Broulik On Sept. 3, 2014, 4:16 nachm., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119916/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2014, 4:16 nachm.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > Since the port of the Button component to QQC Button, the button height > cannot be set anymore, therefore the height of the popup now needs to be > higher when there are three actions. > > > Diffs > ----- > > applets/notifications/package/contents/ui/NotificationPopup.qml abd3ca3 > applets/notifications/plugin/notificationshelper.cpp 53754c5 > > Diff: https://git.reviewboard.kde.org/r/119916/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > Screenshot > > https://git.reviewboard.kde.org/media/uploaded/files/2014/08/23/826a7f99-31f9-47e6-ba58-82ba523f5728__notifications-3actions.png > Screenshot > > https://git.reviewboard.kde.org/media/uploaded/files/2014/09/03/7ef79287-6dbb-4e49-ac44-c5e0a509821c__plasma-notifications_new1.png > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel