[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

Nate Graham  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #16 from Nate Graham  ---
Thanks. Can you file a new bug for that?

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Ben Klopfenstein]

https://bugs.kde.org/show_bug.cgi?id=374864

Ben Klopfenstein  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 CC||benk...@gmail.com
 Resolution|FIXED   |---

--- Comment #15 from Ben Klopfenstein  ---
The fix that was comitted only works for X11 - for wayland, this problem still
exists.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Kai Uwe Broulik]

https://bugs.kde.org/show_bug.cgi?id=374864

Kai Uwe Broulik  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/kwi
   ||n/2ea6fc2abefd810915c38daee
   ||95fe31205ca741b
   Version Fixed In||5.15.3

--- Comment #14 from Kai Uwe Broulik  ---
Git commit 2ea6fc2abefd810915c38daee95fe31205ca741b by Kai Uwe Broulik.
Committed on 05/03/2019 at 21:59.
Pushed by broulik into branch 'Plasma/5.15'.

[effects/screenshot] Set m_windowMode in screenshotWindowUnderCursor
FIXED-IN: 5.15.3

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

M  +1-0effects/screenshot/screenshot.cpp

https://commits.kde.org/kwin/2ea6fc2abefd810915c38daee95fe31205ca741b

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

--- Comment #13 from Nate Graham  ---
That works very reliably for me. Very nice find. Would you like to submit the
patch, David? :)

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[David Redondo]

https://bugs.kde.org/show_bug.cgi?id=374864

David Redondo  changed:

   What|Removed |Added

 CC||k...@david-redondo.de

--- Comment #12 from David Redondo  ---
I think the cause is that Spectacle calls
org.kde.kwin.Screenshot.screenshotWindowUnderCursor and waits for the
screenshotCreated signal which is never send by KWin. I manually called the
method and also didn't see the signal in dbus-monitor. A quick fix is using
screenshotForWindow as we already grab the Id of the window under the cursor. I
don't know how reliable this is compared to the old approach but I was able to
get a picture of only a right click menu.

diff --git a/src/PlatformBackends/X11ImageGrabber.cpp
b/src/PlatformBackends/X11ImageGrabber.cpp
index 4fbb89f..dc5412d 100644
--- a/src/PlatformBackends/X11ImageGrabber.cpp
+++ b/src/PlatformBackends/X11ImageGrabber.cpp
@@ -585,7 +585,7 @@ void X11ImageGrabber::grabWindowUnderCursor()
 mask |= 1 << 1;
 }

-interface.call(QStringLiteral("screenshotWindowUnderCursor"), mask);
+interface.call(QStringLiteral("screenshotForWindow"),
(quint64)windowUnderCursor, mask);
 return;
 }

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

Nate Graham  changed:

   What|Removed |Added

 CC||adam.golan...@gmail.com

--- Comment #11 from Nate Graham  ---
*** Bug 401891 has been marked as a duplicate of this bug. ***

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

Henrik Fehlauer  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|DUPLICATE   |---

--- Comment #10 from Henrik Fehlauer  ---
Hmh, I think that's very unlikely. The issue in the other bug is that closing
is delegated in a lambda to Plasma (!), and if Plasma fails to call back,
Spectacle stays open forever. He we have a problem in getting an image from
KWin, i.e. we are not even in the code path triggering the notification ("GUI
mode").

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

--- Comment #9 from Nate Graham  ---
I was thinking that the root cause is likely the same.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

--- Comment #8 from Henrik Fehlauer  ---
> - It works fine with the compositing/KWin code path.
The other way round, of course.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

--- Comment #7 from Henrik Fehlauer  ---
> *** This bug has been marked as a duplicate of bug 389694 ***
Are you sure this is a duplicate?
- It works fine with the compositing/KWin code path.
- The other bug is about closing, here we don't even get a screenshot.
- The other bug is all about notifications, nothing we have here.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

Nate Graham  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|CONFIRMED   |RESOLVED

--- Comment #6 from Nate Graham  ---


*** This bug has been marked as a duplicate of bug 389694 ***

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

Henrik Fehlauer  changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 Ever confirmed|0   |1

--- Comment #5 from Henrik Fehlauer  ---
> When compositing is off, spectacle takes an empty "screenshot"
That's tracked in Bug 390787.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Dr . Chapatin]

https://bugs.kde.org/show_bug.cgi?id=374864

Dr. Chapatin  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

--- Comment #4 from Dr. Chapatin  ---
Confirmed on Arch Linux, spectacle 17.12.2.
When compositing is off, spectacle takes an empty "screenshot" if cursor is on
an pop-up (dolphin context menu, for example).

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=374864

Nate Graham  changed:

   What|Removed |Added

   Severity|normal  |major
 CC||n...@kde.org

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear with compositing enabled

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

Henrik Fehlauer  changed:

   What|Removed |Added

Summary|Using "Capture the current  |Using "Capture the current
   |pop-up only" option causes  |pop-up only" option causes
   |Spectacle to disappear  |Spectacle to disappear with
   ||compositing enabled

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

Henrik Fehlauer  changed:

   What|Removed |Added

 CC||piotr.mierzwin...@gmail.com

--- Comment #3 from Henrik Fehlauer  ---
*** Bug 381046 has been marked as a duplicate of this bug. ***

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear

[Jack]

https://bugs.kde.org/show_bug.cgi?id=374864

Jack  changed:

   What|Removed |Added

 CC||ostroffjh@users.sourceforge
   ||.net

--- Comment #2 from Jack  ---
I'm seeing this with 17.08.3.  Haven't yet tried turning off compositing.

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

[Spectacle] [Bug 374864] Using "Capture the current pop-up only" option causes Spectacle to disappear

[Henrik Fehlauer]

https://bugs.kde.org/show_bug.cgi?id=374864

Henrik Fehlauer  changed:

   What|Removed |Added

 CC||rk...@lab12.net

--- Comment #1 from Henrik Fehlauer  ---
I noticed this too with Spectacle 17.12.1 and Qt 5.10.1.

However, turning compositing off will capture something instead of getting
stuck. Unfortunately it's not the area of the screen I was interested in.

See also Bug 371768.

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