----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120232/#review66667 -----------------------------------------------------------
applets/notifications/package/contents/ui/NotificationDelegate.qml <https://git.reviewboard.kde.org/r/120232/#comment46469> Not needed. Default position is 0,0 applets/notifications/package/contents/ui/NotificationDelegate.qml <https://git.reviewboard.kde.org/r/120232/#comment46468> Don't set both left and right anchors and the width at the same time applets/notifications/package/contents/ui/NotificationDelegate.qml <https://git.reviewboard.kde.org/r/120232/#comment46465> rightMargin without a right anchor does nothing applets/notifications/package/contents/ui/NotificationDelegate.qml <https://git.reviewboard.kde.org/r/120232/#comment46466> The label wraps, right? That would probably overlap the action buttons, if any. applets/notifications/package/contents/ui/NotificationDelegate.qml <https://git.reviewboard.kde.org/r/120232/#comment46467> Can/Why should there be links in the title? - Kai Uwe Broulik On Sept. 16, 2014, 1:18 nachm., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120232/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2014, 1:18 nachm.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > Basically this copies the style of the floating popup into the notification > history popup. See screenshot. > > > Diffs > ----- > > applets/notifications/package/contents/ui/NotificationDelegate.qml 6c447de > > Diff: https://git.reviewboard.kde.org/r/120232/diff/ > > > Testing > ------- > > Tested with all variety of notifications with and without actions. All looks > good. > > > File Attachments > ---------------- > > screenshot > > https://git.reviewboard.kde.org/media/uploaded/files/2014/09/16/8d9b07f9-03a6-44f7-86d9-cbaab411c116__plasma-notifications_new2.png > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
