flameshot-org / flameshot

Powerful yet simple to use screenshot software :desktop_computer: :camera_flash:
https://flameshot.org
GNU General Public License v3.0
25.12k stars 1.61k forks source link

2nd Screen gets doubled when taking a screenshot #3525

Open sleepylinde opened 9 months ago

sleepylinde commented 9 months ago

Flameshot Version

V12.1.0

Installation Type

Linux, MacOS, or Windows Package manager (apt, pacman, eopkg, choco, brew, ...)

Operating System type and version

Endeavouros on Linux Zen 6.7.8

Description

EOS updated today to KDE6 wayland. Whenever I try to take a screenshot flameshot duplicates my second screen over to my main monitor and then only allows the duplication to be screenshotable.

Steps to reproduce

1) be on KDE6 2) run flameshot 3) take a screenshot

Screenshots or screen recordings

image

System Information

System: Host: linde Kernel: 6.7.8-zen1-1-zen arch: x86_64 bits: 64 Desktop: KDE Plasma v: 6.0.1 Distro: EndeavourOS Graphics: Device-1: AMD Navi 32 [Radeon RX 7700 XT / 7800 XT] driver: amdgpu v: kernel Display: wayland server: X.org v: 1.21.1.11 with: Xwayland v: 23.2.4 compositor: kwin_wayland driver: X: loaded: amdgpu unloaded: modesetting,radeon dri: radeonsi gpu: amdgpu resolution: 1: 1920x1080 2: 1920x1080 API: EGL v: 1.5 drivers: radeonsi,swrast platforms: wayland,x11,surfaceless,device API: OpenGL v: 4.6 compat-v: 4.5 vendor: amd mesa v: 24.0.2-arch1.1 renderer: AMD Radeon RX 7700 XT (radeonsi navi32 LLVM 16.0.6 DRM 3.57 6.7.8-zen1-1-zen) API: Vulkan v: 1.3.279 drivers: radv surfaces: xcb,xlib,wayland

weoses commented 8 months ago

I have same issue

I dug into flameshot sources and figure out that flameshot use one maximized window to display screenshot editor overlay. And whole desktop = one maximized window

When i change showFullScreen() to show() (flameshot.cpp:127), overlay window locates at random position and covers only 2/3 of workspace. And there are some another problems with sizing, i am not an qt developer.. I think on wayland app can not locate own top window on screen (i read it in qt docs - https://doc.qt.io/qt-6/application-windows.html#wayland-peculiarities). Spectacle for example use one window per screen

But i have a workaround Just define this window rules (place at position your workspace size, eg i have 2x1920x1080 = 3840x1080) image

Plasma 6.0.1-1, Arch linux

sleepylinde commented 8 months ago

Thank you for this workaround! Unsure if I should close the issue on this now.

joaociocca commented 8 months ago

huh, I saw another issue that looked similar and jumped on it... but this one seems exactly like the problem I have mentioned in https://github.com/flameshot-org/flameshot/issues/3528#issuecomment-1989000210

sleepylinde commented 8 months ago

@joaociocca going into the settings as weoses described fixed it for me, give it a try

joaociocca commented 8 months ago

ok, I had to make it differently, with the size, but it worked! nice workaround and I actually overlooked it, sorry. Thanks @sleepylinde ! <3

The size here had to be different because of how the displays are arranged. But this capture was already made with it!

image

joaociocca commented 8 months ago

Forgot to mention here, while the workaround works, it takes a serious hit on performance...