[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2020-06-05 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=412186

--- Comment #5 from Méven Car  ---
Git commit dec0e6cef23ac35754aa537db6d7ace42b23ca2a by Méven Car.
Committed on 05/06/2020 at 09:10.
Pushed by meven into branch 'master'.

Refactor Spectacle to run in single instance mode with KDBusService::Unique

Summary:
Since Spectacle got DBus activation support in
09cd11881d828da35c46c48da79f2d988e6a78cc, it has been run in multiple instance
mode.
But it has been shaddowed by using qbus
ee862d161a480408338d00b8826c915f7a97575c in the desktop file to mimic singleton
behavior.

This refactors Spectacle to run in true single instance mode using
KDBusService::Unique.
This is to allow Spectacle to have its executable in its Exec desktop Entry, to
allow KWin to match the executable to the service file, itself to allow
screenshots under Wayland without intermediate click.

The listed bugs are fixed incidently except for 414739 where a simple fix was
added in screenShotUpdated.
Related: bug 420477, bug 414739

Test Plan:
Start spectacle, alternatively, from shortcut, command line.
Take screenshots, from alternative ways.

There is always at most one Spectacle window unless launched with
`--new-instance`.

Reviewers: davidre, ngraham, bport, davidedmundson, apol

Reviewed By: davidre

Subscribers: broulik, anthonyfieroni, #spectacle

Tags: #spectacle

Differential Revision: https://phabricator.kde.org/D29487

M  +2-2desktop/org.kde.spectacle.desktop.cmake
M  +30   -89   src/Main.cpp
M  +170  -105  src/SpectacleCore.cpp
M  +8-10   src/SpectacleCore.h
M  +5-10   src/SpectacleDBusAdapter.cpp
M  +0-31   src/SpectacleDBusAdapter.h

https://invent.kde.org/graphics/spectacle/commit/dec0e6cef23ac35754aa537db6d7ace42b23ca2a

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2020-05-11 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=412186

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||20.04.1

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2020-05-11 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=412186

David Redondo  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
  Latest Commit||https://commits.kde.org/spe
   ||ctacle/b851ca17ae5788def7c1
   ||81f04af9145ee0bb5e68
 Resolution|--- |FIXED

--- Comment #4 from David Redondo  ---
Git commit b851ca17ae5788def7c181f04af9145ee0bb5e68 by David Redondo.
Committed on 11/05/2020 at 13:29.
Pushed by davidre into branch 'release/20.04'.

Disconnect after receiving screenshot signal

Summary:
Only the Spectacle instance that requested a screenshot will receive it.

Reviewers: meven, #spectacle

Reviewed By: meven

Tags: #spectacle

Differential Revision: https://phabricator.kde.org/D29620

M  +4-0src/Platforms/PlatformXcb.cpp

https://commits.kde.org/spectacle/b851ca17ae5788def7c181f04af9145ee0bb5e68

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2020-05-06 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=412186

Méven Car  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED
 CC||meven@enioka.com

--- Comment #3 from Méven Car  ---
Reproduced when both spectacle windows have capture mode set to active window
and include window titlebar and borders turned on.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2020-04-02 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=412186

teemu.pat...@gmail.com changed:

   What|Removed |Added

 CC||teemu.pat...@gmail.com

--- Comment #2 from teemu.pat...@gmail.com ---
Seems like this happens when the capture mode is set to active window and
include window titlebar and borders is turned on.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 412186] Multiple windows with spectacle take the same screenshot

2019-10-06 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=412186

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

--- Comment #1 from Nate Graham  ---
Cannot reproduce.

-- 
You are receiving this mail because:
You are watching all bug changes.