D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-15 Thread Méven Car
meven removed a dependent revision: D29407: ScreenshotEffect: Use Service Property to authorize screenshot without confirmation. REPOSITORY R309 KService REVISION DETAIL https://phabricator.kde.org/D29406 To: meven, #frameworks, davidedmundson, apol, bport Cc: kde-frameworks-devel,

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-05 Thread Méven Car
meven abandoned this revision. meven added a comment. This not necessary, spectacle should have its executable in its Exec desktop entry. REPOSITORY R309 KService REVISION DETAIL https://phabricator.kde.org/D29406 To: meven, #frameworks, davidedmundson, apol, bport Cc:

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-05 Thread Méven Car
meven added a comment. In D29406#663112 , @apol wrote: > > If I understand correctly, it is necessary to use dbus spectacle in this case, so that dbus can manage the application instance and make sure we end up with only one, whether we launch

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-04 Thread Aleix Pol Gonzalez
apol added a comment. > If I understand correctly, it is necessary to use dbus spectacle in this case, so that dbus can manage the application instance and make sure we end up with only one, whether we launch the app, use a shortcut or launch from command line. I don't think that's a

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-04 Thread Méven Car
meven added a comment. In D29406#662932 , @apol wrote: > This should be documented somewhere, as is it's really confusing. I am not sure where to document this, but agrees this is needed. Will look on existing documentation to find the

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-04 Thread Méven Car
meven added a dependent revision: D29407: ScreenshotEffect: Use Service Property to authorize screenshot without confirmation. REPOSITORY R309 KService REVISION DETAIL https://phabricator.kde.org/D29406 To: meven, #frameworks, davidedmundson, apol, bport Cc: kde-frameworks-devel,

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-04 Thread Aleix Pol Gonzalez
apol added a comment. This should be documented somewhere, as is it's really confusing. I'm not sure it wouldn't be better to have spectacle launched again there instead of going through dbus (which is a change I admittedly don't understand very well). REPOSITORY R309 KService

D29406: Add X-KDE-Original-Executable to Applications properties

2020-05-04 Thread Méven Car
meven created this revision. meven added reviewers: Frameworks, davidedmundson, apol, bport. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. meven requested review of this revision. REVISION SUMMARY Sometimes Exec entries are replaced by qdbus invocation