[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-02-17 Thread d_avery
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #23 from d_avery  ---
My mistake. I forgot I had put kguidaddons in IgnorePkg and it did not actually
update when I ran a full system upgrade. 5.103 fixes the issue.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-02-15 Thread d_avery
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #22 from d_avery  ---
After updating to kguiaddons 5.103, issue is still present.

KDE Plasma 5.26.90 Wayland session, Arch Linux kernel 6.1.11-arch1-1.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-29 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=463199

Patrick Silva  changed:

   What|Removed |Added

 CC||tus...@kazv.moe

--- Comment #21 from Patrick Silva  ---
*** Bug 464997 has been marked as a duplicate of this bug. ***

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-28 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=463199

Fushan Wen  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/fram |https://invent.kde.org/fram
   |eworks/kguiaddons/commit/dc |eworks/kguiaddons/commit/e1
   |b254dc6b4e5b76e04a5ee8b4db8 |95913d37f8ff79a14351c029952
   |177aff10d29 |830394e7057

--- Comment #20 from Fushan Wen  ---
Git commit e195913d37f8ff79a14351c029952830394e7057 by Fushan Wen, on behalf of
David Redondo.
Committed on 29/01/2023 at 05:14.
Pushed by fusionfuture into branch 'cherry-pick-dcb254dc'.

waylandclipboard: roundtrip to get accurate focus state

The current approach is too complicated and not correct anyway.
To summarize QGuiApplication::focusWindow is delayed so the focus
watcher was introduced, however when the window is hidden and
setMimeData called before Wayland events have been processed,
then focusWatcher is naturally out of sync, so the check
was complicated to work around this state. However the same
issue arises when the window is deleted before calling
setMimeData.
Instead of introducing more complicated checks and workaround
roundtrip so the focusWatcher state is up to date before setting
the clipboard.
FIXED-IN: 5.103


(cherry picked from commit dcb254dc6b4e5b76e04a5ee8b4db8177aff10d29)

M  +8-3src/systemclipboard/waylandclipboard.cpp

https://invent.kde.org/frameworks/kguiaddons/commit/e195913d37f8ff79a14351c029952830394e7057

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-27 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=463199

David Edmundson  changed:

   What|Removed |Added

   Version Fixed In||5.103
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/fram
   ||eworks/kguiaddons/commit/dc
   ||b254dc6b4e5b76e04a5ee8b4db8
   ||177aff10d29
 Resolution|--- |FIXED

--- Comment #19 from David Edmundson  ---
Git commit dcb254dc6b4e5b76e04a5ee8b4db8177aff10d29 by David Edmundson, on
behalf of David Redondo.
Committed on 27/01/2023 at 22:26.
Pushed by davidedmundson into branch 'master'.

waylandclipboard: roundtrip to get accurate focus state

The current approach is too complicated and not correct anyway.
To summarize QGuiApplication::focusWindow is delayed so the focus
watcher was introduced, however when the window is hidden and
setMimeData called before Wayland events have been processed,
then focusWatcher is naturally out of sync, so the check
was complicated to work around this state. However the same
issue arises when the window is deleted before calling
setMimeData.
Instead of introducing more complicated checks and workaround
roundtrip so the focusWatcher state is up to date before setting
the clipboard.
FIXED-IN: 5.103

M  +8-3src/systemclipboard/waylandclipboard.cpp

https://invent.kde.org/frameworks/kguiaddons/commit/dcb254dc6b4e5b76e04a5ee8b4db8177aff10d29

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-27 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=463199

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #18 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/frameworks/kguiaddons/-/merge_requests/90

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-27 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #17 from David Redondo  ---
I reproduced now with the release version of spectacle compared to the
rewritten version.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-26 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #16 from matteodinoi...@gmail.com ---
Having same problem on Arch (Kernel 6.1.8, Plasma version 5.26.5, Kde
Frameworks 5.102.0) and downgrading to kguiaddons 5.101 seem to be working
partially (meaning that it now copy to clipboard but using spectacle -brc it
just never stop the command until i use ^C)

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-26 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=463199

matteodinoi...@gmail.com changed:

   What|Removed |Added

 CC||matteodinoi...@gmail.com

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-25 Thread Matt
https://bugs.kde.org/show_bug.cgi?id=463199

Matt  changed:

   What|Removed |Added

 CC||garage...@mailbox.org

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-25 Thread Andrej Halveland
https://bugs.kde.org/show_bug.cgi?id=463199

Andrej Halveland  changed:

   What|Removed |Added

 CC||andrej.h...@gmail.com

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-25 Thread hexchain
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #15 from hexchain  ---
Tried to do some debugging on this issue with some added output to KGuiAddons:
https://gist.github.com/hexchain/06e71deb8b29f661fee646fe0096a499

Spectacle prints the following output after a screenshot is taken (and nothing
is copied to the clipboard):
kb enter
using normal mechanism fw: QWindow(0x0) , fw->isVisible: false , keyboard has
focus: true
kb leave

If I remove the whole "if" block of the "normal mechanism", Spectacle is then
able to copy the image to the clipboard. With WAYLAND_DEBUG the output also
matches what you expected in #c11.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=463199

Nate Graham  changed:

   What|Removed |Added

 CC||amanita+kdeb...@mailbox.org

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

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-23 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=463199

Nate Graham  changed:

   What|Removed |Added

 CC||o.rootedavi...@gmail.com

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

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-19 Thread hexchain
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #12 from hexchain  ---
I honestly have no idea. It still happens even after removing
~/.config/spectaclerc.

Are you running everything from git master? Could it be something other than
KGuiAddons?

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-19 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #11 from David Redondo  ---
Even with these commands copies happen for me. From your wayland logs it looks
to me like you wouldn't have
https://invent.kde.org/frameworks/kguiaddons/-/commit/0fbb0aa524adbd077b2ba7716218d4c6f042bed0
but it should be included in 5.102:

Window is hidden (surface destroyed) but data_device is used

[3695470.952] wl_pointer@26.frame()
[3695471.113]  -> xdg_toplevel@41.destroy()
[3695471.116]  -> xdg_surface@40.destroy()
[3695471.122]  -> wl_disp...@1.sync(new id wl_callback@53)
[3695471.124]  -> wl_surface@37.destroy()
[3695471.614]  -> org_kde_plasma_surface@34.destroy()
[3695472.136]  -> wl_shm_pool@51.destroy()
[3695472.141]  -> wl_buffer@52.destroy()
[3695472.696]  -> wl_shm_pool@42.destroy()
[3695472.700]  -> wl_buffer@43.destroy()
[3695472.732]  -> wl_data_device_manager@9.create_data_source(new id
wl_data_source@54)
[3695474.937]  -> wl_data_source@54.offer("application/x-qt-image")
[3695474.941]  -> wl_data_source@54.offer("x-kde-force-image-copy")
[3695474.942]  -> wl_data_source@54.offer("image/png")
[...]
[3695474.956]  -> wl_data_device@10.set_selection(wl_data_source@54, 26798)

[1946768.478] wl_pointer@20.frame()
[1946770.191]  -> xdg_toplevel@35.destroy()
[1946770.208]  -> xdg_surface@34.destroy()
[1946770.227]  -> wl_disp...@1.sync(new id wl_callback@50)
[1946770.240]  -> wl_surface@31.destroy()
[1946770.673]  -> org_kde_plasma_surface@28.destroy()
[1946771.791]  -> wl_shm_pool@46.destroy()
[1946771.812]  -> wl_buffer@47.destroy()
[1946772.872]  -> wl_shm_pool@41.destroy()
[1946772.886]  -> wl_buffer@42.destroy()
[1946774.022]  -> wl_shm_pool@36.destroy()
[1946774.042]  -> wl_buffer@37.destroy()
[1946775.128]  -> wl_shm_pool@44.destroy()
[1946775.144]  -> wl_buffer@45.destroy()
[1946775.967]  -> wl_data_device_manager@9.create_data_source(new id
wl_data_source@51)
[1946776.192]  -> wl_data_source@51.offer("application/x-qt-image")
[1946776.223]  -> wl_data_source@51.offer("x-kde-force-image-copy")
[1946776.232]  -> wl_data_source@51.offer("image/png")
[...]
[1946776.407]  -> wl_data_device@10.set_selection(wl_data_source@51, 27682)

While on my machine happens what I would expect, no window so data_control is
used:
[1118803.392] wl_keybo...@3.key(2533, 3267142, 28, 1)
[1118804.192]  -> xdg_toplevel@40.destroy()
[1118804.213]  -> xdg_surface@39.destroy()
[1118804.256]  -> wl_disp...@1.sync(new id wl_callback@77)
[1118804.272]  -> wl_surface@35.destroy()
[1118804.324]  -> org_kde_kwin_appmenu@69.release()
[1118806.243]  -> zwlr_data_control_manager_v1@24.create_data_source(new id
zwlr_data_control_source_v1@81)
[1118806.261]  ->
zwlr_data_control_source_v1@81.offer("application/x-qt-image")
[1118806.269]  ->
zwlr_data_control_source_v1@81.offer("x-kde-force-image-copy")
[1118806.413]  -> zwlr_data_control_source_v1@81.offer("image/png")
[...]
[1118806.586]  ->
zwlr_data_control_device_v1@27.set_selection(zwlr_data_control_source_v1@81)

I am bit of a loss at the moment why it would happen

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-18 Thread hexchain
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #10 from hexchain  ---
Created attachment 155396
  --> https://bugs.kde.org/attachment.cgi?id=155396=edit
Wayland debug logs for running spectacle -ircb

My steps are the same as in #c7 (except I added an additional -i flag). Here is
my Wayland debug logs.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #9 from Nate Graham  ---
Can confirm that it is now working for me with current git master as well.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread Gabriel Garcia
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #8 from Gabriel Garcia  ---
Created attachment 155375
  --> https://bugs.kde.org/attachment.cgi?id=155375=edit
logs for running spectacle -rcb

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread Gabriel Garcia
https://bugs.kde.org/show_bug.cgi?id=463199

Gabriel Garcia  changed:

   What|Removed |Added

 CC||gabrielgar...@gmail.com

--- Comment #7 from Gabriel Garcia  ---
(In reply to David Redondo from comment #6)
> - could you give more detailed steps, maybe there is a difference in my
> testing?
Launch spectacle -rbc either through the console or through a shortcut I've
created through system settings. Select a rectangular region, I get a
notification saying it's copied to the clipboard, it's not copied to the
clipboard.

- provide me the output of `WAYLAND_DEBUG=1 spectacle` of taking jsut one
screenshot with that setting.
Attached is the output of `WAYLAND_DEBUG=1 spectacle -rbc`

Launching Spectacle gui, taking a screenshot, and manually clicking the copy to
clipboard button works. 
Launching Spectacle with the background option + copy to clipboard + any other
option except the rectangular one works.
Launching Spectacle with the background option + copy to clipboard +
rectangular selection, doesn't work.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #6 from David Redondo  ---
If you can
- could you give more detailed steps, maybe there is a difference in my
testing?
- provide me the output of `WAYLAND_DEBUG=1 spectacle` of taking jsut one
screenshot with that setting.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #5 from David Redondo  ---
I've updated my system and can't reproduce anymore.

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-17 Thread David Redondo
https://bugs.kde.org/show_bug.cgi?id=463199

--- Comment #4 from David Redondo  ---
Seems for me the delay settings play a role
- no delay no copy is happening
- 1s or more copy works

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

[frameworks-kguiaddons] [Bug 463199] "After taking screenshot: Copy to clipboard" does not work anymore on Wayland

2023-01-16 Thread Noah Davis
https://bugs.kde.org/show_bug.cgi?id=463199

Noah Davis  changed:

   What|Removed |Added

Version|unspecified |5.102.0
Product|Spectacle   |frameworks-kguiaddons
  Component|General |general

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