[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 Nate Graham changed: What|Removed |Added See Also||https://bugs.kde.org/show_b ||ug.cgi?id=440226 -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 Nate Graham changed: What|Removed |Added Status|RESOLVED|VERIFIED --- Comment #27 from Nate Graham --- Verified. Woohoo! -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 Nate Graham changed: What|Removed |Added Version Fixed In|5.19|20.12 Resolution|--- |FIXED Latest Commit|https://cgit.kde.org/kwin.g |https://invent.kde.org/grap |it/commit/?id=66898e7f46f2d |hics/spectacle/commit/4a3cb |e13d7ba34f62fb85daf5f21e161 |5140cc58d73b1f0f0083d7f24f9 ||f3a40b2a Status|CONFIRMED |RESOLVED --- Comment #26 from Nate Graham --- Git commit 4a3cb5140cc58d73b1f0f0083d7f24f9f3a40b2a by Nate Graham, on behalf of Méven Car. Committed on 20/11/2020 at 14:21. Pushed by ngraham into branch 'release/20.12'. Support Wayland rectangular selection HiDpi When the drawn rectangle is on a single screen, the output image uses the screen native resolution. When the drawn rectangle is on screens with multiple scale factor, the output image is upscaled to correspond to the screen with the hightest scale factor to keep the image undistorted. Adds also a "scaled" full screen mode, to capture screens content downscaled to screens virtualsize allowing for undistorted content albeit loosing details. Requires Plasma 5.19.80+ Related: bug 420863 FIXED-IN: 20.12 M +17 -9src/Gui/KSWidget.cpp M +7-1src/Platforms/Platform.h M +133 -21 src/Platforms/PlatformKWinWayland.cpp M +11 -2src/Platforms/PlatformKWinWayland.h M +1-1src/Platforms/PlatformNull.cpp M +15 -1src/Platforms/PlatformXcb.cpp A +41 -0src/QuickEditor/ComparableQPoint.h [License: LGPL (v2+)] M +274 -91 src/QuickEditor/QuickEditor.cpp M +15 -3src/QuickEditor/QuickEditor.h M +2-1src/SpectacleCommon.h M +35 -11 src/SpectacleCore.cpp M +1-0src/SpectacleCore.h https://invent.kde.org/graphics/spectacle/commit/4a3cb5140cc58d73b1f0f0083d7f24f9f3a40b2a -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 Nate Graham changed: What|Removed |Added See Also||https://bugs.kde.org/show_b ||ug.cgi?id=368939 -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 ruruoli changed: What|Removed |Added CC||maniikarab...@protonmail.ch -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #25 from Claudius Ellsel --- (In reply to Marvin Hagemeister from comment #22) > As a user I'd expect screenshots to be always taken in the native > resolution, regardless of whether I have multiple screens attached or not. > I'd be confused if there is any downscaling happening behind the scenes. > When doing a screenshot the point is to capture everything as is, and > anything else seems to take away from that. The problem with capturing everything "as is" is that you cannot really do that. From my understanding on multiscreen setups one large image will be created containing the content of all screens. With scaling enabled, you could either use the native resolution resulting in overlapping windows not matching for example (while they match perfectly when just viewing the screens because both have different DPIs). This is displayed in the post on Phabricator I linked to in comment #21. To make things look as the user sees them, one has to use down- or upscaling in order to get multiple screens on one single image. I just want to make sure you are aware of the facts, since I suspect you did not know that before? I think you agree that we at least need an option to scale in such circumstances so the user can get properly aligned result if they want to. I think this also is the reason, why the suggestion from comment #23 will not work, since all screens are put into one single image. (In reply to Méven Car from comment #24) > (In reply to Marvin Hagemeister from comment #22) > So I have opposite opinions, a setting/option seems to be necessary. > It will be wayland only, not-native by default (current screen is always > native now so it will be a fast path). Sounds good to me! -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #24 from Méven Car --- (In reply to Marvin Hagemeister from comment #22) > As a user I'd expect screenshots to be always taken in the native > resolution, regardless of whether I have multiple screens attached or not. > I'd be confused if there is any downscaling happening behind the scenes. > When doing a screenshot the point is to capture everything as is, and > anything else seems to take away from that. So I have opposite opinions, a setting/option seems to be necessary. It will be wayland only, not-native by default (current screen is always native now so it will be a fast path). (In reply to Christoph Feck from comment #23) > It would help if snapshots were saved with DPI information. A viewer could > then decide if the images need to be scaled. That's not really possible for multiple screens with multiple scale factors... -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #23 from Christoph Feck --- It would help if snapshots were saved with DPI information. A viewer could then decide if the images need to be scaled. -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #22 from Marvin Hagemeister --- As a user I'd expect screenshots to be always taken in the native resolution, regardless of whether I have multiple screens attached or not. I'd be confused if there is any downscaling happening behind the scenes. When doing a screenshot the point is to capture everything as is, and anything else seems to take away from that. -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #21 from Claudius Ellsel --- (In reply to Méven Car from comment #19) > https://invent.kde.org/plasma/kwin/-/merge_requests/194 will fix the > "Current screen" case. Thanks for the quick fix! > But for muliti-screen cases should we always take screenshots in native (not > downscaled) ? > For multiscreen screenshots, this results in distorted results. > Or Should I expose a choice, or default to non-native for multiscreen with a > setting ? I guess with "distorted results" you are referring to problems like shown in https://phabricator.kde.org/D25427#648841 for example, where the screenshot taken looks completely different to what the user sees? I like the idea to have a choice between native resolution and downscaled for multi-screen cases. Or maybe always downscale and only do native screenshots on multi-screen setups when the user does a "current screen" screenshot. But I guess having an option would not hurt if it does not make the interface too complicated for the user. Also it might be good to have some info text for each option noting the downsides of the selected one (either downscaled or looking unproportional). -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #20 from Méven Car --- Git commit 9a8b6f730d8d516594f159dbb41aa1357d65055d by Méven Car. Committed on 19/08/2020 at 10:23. Pushed by meven into branch 'master'. Make ScreenShotEffect::screenshotScreen return native screen sized images Aka not-downscaled images. M +2-0effects/screenshot/screenshot.cpp https://invent.kde.org/plasma/kwin/commit/9a8b6f730d8d516594f159dbb41aa1357d65055d -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 --- Comment #19 from Méven Car --- (In reply to Claudius Ellsel from comment #18) > (In reply to Méven Car from comment #16) > > You are right Patrick, this depends on a spectacle patch, and change in : > > https://phabricator.kde.org/D29282#inline-169560 > > This linked patch looks to me like it is only (or mostly?) about > multi-screen. Might there be a way to make this work on single screen before > handling the special issues with multi-screen? Currently the conversation > there looks like it might be either blocked currently or need some > additional work to incorporate the suggestion from David Edmundson there. The right commit relating to this is https://phabricator.kde.org/D29010 But https://phabricator.kde.org/D29122 introduced a change since that introduced a regression. https://invent.kde.org/plasma/kwin/-/merge_requests/194 will fix the "Current screen" case. But for muliti-screen cases should we always take screenshots in native (not downscaled) ? For multiscreen screenshots, this results in distorted results. Or Should I expose a choice, or default to non-native for multiscreen with a setting ? -- You are receiving this mail because: You are watching all bug changes.
[Spectacle] [Bug 409762] [wayland] Screenshots taken on displays with scaling lack resolution
https://bugs.kde.org/show_bug.cgi?id=409762 Claudius Ellsel changed: What|Removed |Added Summary|[wayland] Screenshots taken |[wayland] Screenshots taken |on High DPI systems lose|on displays with scaling |resolution |lack resolution -- You are receiving this mail because: You are watching all bug changes.
