Review Request 119719: Streamline notifications plasmoid

Kai Uwe Broulik kde at privat.broulik.de
Mon Aug 11 18:55:59 UTC 2014


> On Aug. 11, 2014, 6:37 nachm., Martin Klapetek wrote:
> > > I have no idea what's wrong with the action button.
> > 
> > This broke since the port to QtQuick Controls' Button, not sure what to do about it tho

Well, clearly the button's label does not honor the button's custom height as set by the NotificationsDelegate, as can be easily seen in [1].

[1] http://wstaw.org/w/2Thp/


- Kai Uwe


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119719/#review64296
-----------------------------------------------------------


On Aug. 11, 2014, 6:19 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119719/
> -----------------------------------------------------------
> 
> (Updated Aug. 11, 2014, 6:19 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This streamlines the notification plasmoid a bit.
> 
> - Get rid of hardcoded sizes (was quite cramped here)
> 
> - Get rid of huge "Transfers" heading. It was the same size as the systray title and added little value
> 
> I have no idea what's wrong with the action button.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/Jobs.qml 2f5a88e 
>   applets/notifications/package/contents/ui/NotificationDelegate.qml 0b97d77 
> 
> Diff: https://git.reviewboard.kde.org/r/119719/diff/
> 
> 
> Testing
> -------
> 
> Yup.
> 
> 
> File Attachments
> ----------------
> 
> Notifications (before)
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/08/11/da7f1f4e-df8f-4eda-aca8-1c4fcc6fdeda__notifyold.png
> Notifications (after)
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/08/11/1064cc3d-8ae0-4267-ae6e-1643f8a0a23a__notifynew.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140811/9d2f5d6b/attachment.html>


More information about the Plasma-devel mailing list