[Differential] [Accepted] D2218: New logic for screen numbers in plasmashell
davidedmundson (David Edmundson)
noreply at phabricator.kde.org
Mon Jul 25 11:42:39 UTC 2016
davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
davidedmundson added a comment.
This revision is now accepted and ready to land.
Ship it.
INLINE COMMENTS
> panelview.cpp:1043
>
> void PanelView::screenDestroyed(QObject* )
> {
we can just kill this entire method
> panelview.h:89
> */
> - Q_PROPERTY(QScreen *screen READ screen WRITE setScreen NOTIFY screenChangedProxy)
> + Q_PROPERTY(QScreen *screen READ screenToFollow WRITE setScreenToFollow NOTIFY screenToFollowChanged)
>
I think even though it means a change in plasma-desktop, we should change this property name.
We were previously shadowing an existing property, but doing a hack to also expose a write method.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
BRANCH
mart/screenConnectorManagement
REVISION DETAIL
https://phabricator.kde.org/D2218
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: mart, #plasma, davidedmundson
Cc: lbeltrame, davidedmundson, rwooninck, graesslin, plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160725/90cffd3b/attachment.html>
More information about the Plasma-devel
mailing list