> On March 24, 2015, 5:22 p.m., Kai Uwe Broulik wrote:
> > Nice!
> > 
> > I'm not sure whether a global setting should be in the applet configuration 
> > but fair enough. The "Default" keeps the previous behavior where it sort-of 
> > follows your panel?

Yes.


> On March 24, 2015, 5:22 p.m., Kai Uwe Broulik wrote:
> > applets/notifications/package/contents/ui/configNotifications.qml, line 64
> > <https://git.reviewboard.kde.org/r/122842/diff/4/?file=356476#file356476line64>
> >
> >     Emitting a signal inside a property binding looks wrong to me.

Well because the "screenPosition" property is not part of the regular config 
properties, something needs to tell the config dialog that things have changed 
and that it should enable the "Apply" button. Since the binding /requires/ a 
function, I thought why not?


- Martin


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


On March 24, 2015, 1:19 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122842/
> -----------------------------------------------------------
> 
> (Updated March 24, 2015, 1:19 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> 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
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to