Review Request 122948: Use fontSizeMode Text.Fit for NotificationIcon
Kai Uwe Broulik
kde at privat.broulik.de
Sat Mar 14 23:04:31 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122948/
-----------------------------------------------------------
(Updated March 14, 2015, 11:04 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and KDE Usability.
Changes
-------
Submitted with commit f7f12485aab02785bd7bfb880deba39615c53f02 by Kai Uwe Broulik to branch Plasma/5.2.
Repository: plasma-workspace
Description
-------
https://www.kde.org/announcements/plasma-5.2/full-screen.png
There's nothing more to say. (Also use Connections rather than temp property)
Diffs
-----
applets/notifications/package/contents/ui/NotificationIcon.qml 1449c19
Diff: https://git.reviewboard.kde.org/r/122948/diff/
Testing
-------
Font now automatically scales with the icon. Alignment can still be a little bit off at times but properly centering text is a horrible task, also given the frame and everything else is themable. We lack the triangle indicators now, though.
File Attachments
----------------
Before in tray
https://git.reviewboard.kde.org/media/uploaded/files/2015/03/14/38fc6d1c-a1f7-4ace-8df3-5803516862f8__notificationiconbefore.png
After in tray
https://git.reviewboard.kde.org/media/uploaded/files/2015/03/14/4cf0dd62-7d14-4f9c-81aa-045d5f02b709__notificationiconafter.png
On Desktop
https://git.reviewboard.kde.org/media/uploaded/files/2015/03/14/b5111380-e5f2-4f45-93e5-fb84da26b711__notificationiconafter1.png
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150314/1ef3df00/attachment.html>
More information about the Plasma-devel
mailing list