Review Request 122842: Allow selecting notifications position on screen

Martin Klapetek martin.klapetek at gmail.com
Mon Apr 6 18:09:23 UTC 2015


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

(Updated April 6, 2015, 6:09 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and KDE Usability.


Changes
-------

Submitted with commit 9429c845ae3fee1d65b026b2d496aae1305db232 by Martin Klapetek to branch master.


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/20150406/48fc8a90/attachment.html>


More information about the Plasma-devel mailing list