Bug 426107 - When annotating, viewport inappropriately shows zoomed-in full size image
Summary: When annotating, viewport inappropriately shows zoomed-in full size image
Status: CONFIRMED
Alias: None
Product: Spectacle
Classification: Applications
Component: General (show other bugs)
Version: 23.04.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Boudhayan Gupta
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-09-02 18:10 UTC by Nate Graham
Modified: 2023-12-13 20:59 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screen recording (2.18 MB, video/mp4)
2020-09-02 18:10 UTC, Nate Graham
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2020-09-02 18:10:26 UTC
Created attachment 131364 [details]
Screen recording

Right now the annotation viewport shows the image at 100% size in a scrollable viewport. This is a sub-optimal UX, especially for HiDPI where the 100% size image is much larger than the viewport so you can only ever see a tiny fraction of it. See attached screen recording.

It would be nicer if the annotator viewport showed a scaled representation of the image that fit completely within its area.
Comment 1 mateMat 2021-02-02 22:29:46 UTC
I confirm this bug.
If you keep clicking on the annotation in and out icon, the picture zooms again and again.
My screen is 2k, DPI ratio 1.5.

Operating System: KDE neon 5.20
KDE Plasma Version: 5.20.5
KDE Frameworks Version: 5.78.0
Qt Version: 5.15.2
Kernel Version: 5.4.0-64-generic
Comment 2 Antonio Prcela 2022-03-02 22:22:42 UTC
I've added in upstream a button to fit image into current view:
https://github.com/ksnip/kImageAnnotator/pull/281

and then there is https://bugs.kde.org/show_bug.cgi?id=429833
with MR: https://invent.kde.org/graphics/spectacle/-/merge_requests/127

Or you got an idea/suggestion to do it differently?
Comment 3 Nate Graham 2022-03-02 22:53:03 UTC
Oh, that's great! Could we make the image fit in whatever size the window is rather than show full size by default? That would probably be sufficient to fix the issue, independently of your proposed fix for Bug 429833.