Review Request 124151: Add a default icon to notifications popup

[Martin Klapetek]

> On June 22, 2015, 7:54 p.m., Kai Uwe Broulik wrote:
> > what about just freeing the space instead? i'm find with this solution though

I didn't do that for two reasons - apps/commands setting no icon would just look out of place with their notification. Second is that the icon plays an important part in the layout - it sets min height as well as having anchors attached to it. It could be changed but would make the layouting more complex imo, which is not worth it for the reason #1.


- Martin


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


On June 22, 2015, 7:51 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124151/
> -----------------------------------------------------------
> 
> (Updated June 22, 2015, 7:51 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Currently there's none and if there's just a summary text ($ notify-send text), the notification popup looks strange. Adding the default icon makes it look more consistent and not totally-out-of-place-command-line-notification.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/Notifications.qml 13be7d4 
> 
> Diff: https://git.reviewboard.kde.org/r/124151/diff/
> 
> 
> Testing
> -------
> 
> See screenshot
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/06/22/0d8a0ee1-af00-410c-80f1-79190f51daf5__notification-default-icon.png
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150622/28ddc4ed/attachment-0001.html>