D3769: expose immutable through dbus

2018-08-28 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. scripts now run when we're immutable, so I think this isn't needed REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3769 To: mvou

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mvourlakos (Michail Vourlakos)
mvourlakos added a comment. In https://phabricator.kde.org/D3769#70551, @davidedmundson wrote: > > why? isnt kwayland independent of plasmashell? > > Sort of. > > Wayland core protocols (by design) do not allow a window to position itself, as a security measure, > Plasma needs

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mvourlakos (Michail Vourlakos)
mvourlakos added a comment. I have responded in the mailing list into the topic "Is it possible to add a different panel implementation for plasma?" in order to justify more the design decision taken. We can continue there if you want to. I am not negative to reconsider and I am here to

[Differential] [Updated] D3769: expose immutable through dbus

2016-12-21 Thread davidedmundson (David Edmundson)
davidedmundson added a comment. > why? isnt kwayland independent of plasmashell? Sort of. Wayland core protocols (by design) do not allow a window to position itself, as a security measure, Plasma needs to so, as you've pointed out KWayland contains a protocol to speak to kwin to

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mart (Marco Martin)
mart added a comment. In https://phabricator.kde.org/D3769#70547, @mvourlakos wrote: > > note that if it becomes an outside application there is no way it would ever work under wayland > > why? isnt kwayland independent of plasmashell? the plasma wayland protocol, that has t

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mvourlakos (Michail Vourlakos)
mvourlakos added a comment. In https://phabricator.kde.org/D3769#70526, @mart wrote: > In https://phabricator.kde.org/D3769#70449, @mvourlakos wrote: > > > It's for Now Dock's next version, a discussion can be found here: > > https://phabricator.kde.org/D3754 in next version Now Doc

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mart (Marco Martin)
mart added a comment. no, this kind of stuff should not be ever exported trough dbus REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3769 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mvourlakos, #plasma, davidedmundso

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-21 Thread mart (Marco Martin)
mart added a comment. In https://phabricator.kde.org/D3769#70449, @mvourlakos wrote: > It's for Now Dock's next version, a discussion can be found here: > https://phabricator.kde.org/D3754 in next version Now Dock will be a Dock > outside plasmashell but I want the user to have a uni

Re: [Differential] [Commented On] D3769: expose immutable through dbus

2016-12-20 Thread Michail Vourlakos
It's for Now Dock's next version, a discussion can be found here: https://phabricator.kde.org/D3754 in next version Now Dock will be a Dock outside plasmashell but I want the user to have a unified experience with plasmashell. Unlocking widgets in plasma unlocks them also for my Dock and in reverse

[Differential] [Commented On] D3769: expose immutable through dbus

2016-12-20 Thread apol (Aleix Pol Gonzalez)
apol added a comment. It's a bit scary that applications can randomly change this state, no? What's the use-case? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3769 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mvou

[Differential] [Updated] D3769: expose immutable through dbus

2016-12-20 Thread mvourlakos (Michail Vourlakos)
mvourlakos added a reviewer: davidedmundson. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3769 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mvourlakos, #plasma, davidedmundson Cc: plasma-devel, lesliezhai, ali-mohamed,

[Differential] [Request, 29 lines] D3769: expose immutable through dbus

2016-12-20 Thread mvourlakos (Michail Vourlakos)
mvourlakos created this revision. mvourlakos added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY --expose immutable and signaling immutability changes through dbus in order to be used from applicati