Review Request 122842: Allow selecting notifications position on screen

[Thomas Pfeiffer]

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


Thank you for adding usability!

If the checkbox is disabled, the whole visualization of the screen should be greyed out / desaturated, because it is simply not relevant in that case.

>From the design perspective it would make sense to completely harmonize the representation of the screen with the screen edge KCM (except for round vs. square buttons). I'll let our visual designers decide whether the background should be light or dark, as long as it's the same in both cases.

Otherwise, it's fine. Very nice feature!

- Thomas Pfeiffer


On April 1, 2015, 5:30 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122842/
> -----------------------------------------------------------
> 
> (Updated April 1, 2015, 5:30 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Bugs: 344841
>     https://bugs.kde.org/show_bug.cgi?id=344841
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Allows to select each corner and top or bottom edges of the screen.
> 
> The radio selection thing is currently a bit ugly and I'd like to ask for suggestions on it (I originally wanted to add VDG here but there's no vdg group (?) so I'll post in the forums).
> 
> 
> Diffs
> -----
> 
>   applets/notifications/CMakeLists.txt 113d731 
>   applets/notifications/lib/CMakeLists.txt PRE-CREATION 
>   applets/notifications/lib/notificationsapplet.h PRE-CREATION 
>   applets/notifications/lib/notificationsapplet.cpp PRE-CREATION 
>   applets/notifications/package/contents/ui/Notifications.qml 4312774 
>   applets/notifications/package/contents/ui/ScreenPositionSelector.qml PRE-CREATION 
>   applets/notifications/package/contents/ui/configNotifications.qml cda9ac9 
>   applets/notifications/package/metadata.desktop 333fa2c 
>   applets/notifications/plugin/notificationshelper.h ca0b63b 
>   applets/notifications/plugin/notificationshelper.cpp cd8a5a5 
> 
> Diff: https://git.reviewboard.kde.org/r/122842/diff/
> 
> 
> Testing
> -------
> 
> All positions work
> 
> 
> File Attachments
> ----------------
> 
> Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/03/06/f80f9345-3d5f-49b4-83d7-16ecbfbc607a__notification_pos.png
> New Screenshot
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/03/11/e1fd3405-b008-4d6e-90c7-2000ab011008__notifications_config.png
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150401/5b95febb/attachment.html>