Bug 401504 - Crop box should start at visible image region
Summary: Crop box should start at visible image region
Status: CONFIRMED
Alias: None
Product: gwenview
Classification: Applications
Component: general (show other bugs)
Version: 18.08.3
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Gwenview Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-28 13:24 UTC by Greg Lepore
Modified: 2020-02-21 11:28 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of image immediately after selecting Crop (2.05 MB, image/png)
2018-11-28 13:24 UTC, Greg Lepore
Details
Screenshot of image after scrolling up to see lower crop handles. (2.05 MB, image/png)
2018-11-28 13:25 UTC, Greg Lepore
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Lepore 2018-11-28 13:24:42 UTC
Created attachment 116546 [details]
Screenshot of image immediately after selecting Crop

SUMMARY
When selecting Crop from the toolbar, the lower bounds of the crop box are off the screen, requiring a scroll up to begin to utilize those crop handles.

See attached crop1 for the screen immediately after selecting Crop, and crop2 image for the image after scrolling up to see the lower crop handles.


STEPS TO REPRODUCE
1. Select Crop tool
2. 
3. 

OBSERVED RESULT
Lower crop handles are off screen

EXPECTED RESULT
Crop handles should be immediately visible.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.14
KDE Plasma Version: 5.14.4
Qt Version: 5.11.2
KDE Frameworks Version: 5.52.0
Kernel Version: 4.15.0-39-generic
OS Type: 64-bit
Processors: 8 × AMD FX(tm)-8350 Eight-Core Processor
Memory: 7.6 GiB of RAM


ADDITIONAL INFORMATION
Comment 1 Greg Lepore 2018-11-28 13:25:49 UTC
Created attachment 116547 [details]
Screenshot of image after scrolling up to see lower crop handles.
Comment 2 Christoph Feck 2018-12-20 00:44:46 UTC
So you expect Gwenview to always zoom-to-fit when starting Crop mode?
Comment 3 Greg Lepore 2018-12-20 02:28:38 UTC
My expected result was for all crop handles to be visible on screen without scrolling after selecting Crop. Nothing about the Zoom level should change.

Generally my work flow for Cropping images is to zoom to the proper level of viewing, so that the entire area I want cropped is visible, and then selecting the Crop tool. I thought this was Gwenview's behavior, but perhaps I was mistaken or the behavior has changed recently.
Comment 4 Christoph Feck 2018-12-20 02:50:23 UTC
Thanks for the update; changing title for clarification.
Comment 5 Peter Mühlenpfordt 2018-12-27 11:19:39 UTC
This is caused by the removal of `onWidgetSlidedIn` in
https://phabricator.kde.org/D11378.
But just re-adding it does not work for Advanced settings mode.
Comment 6 Greg Lepore 2020-02-21 11:28:06 UTC
Bug is still occuring on:

Operating System: KDE neon 5.18
KDE Plasma Version: 5.18.1
KDE Frameworks Version: 5.67.0
Qt Version: 5.14.1
Kernel Version: 4.15.0-76-generic
OS Type: 64-bit
Processors: 8 × AMD FX(tm)-8350 Eight-Core Processor
Memory: 7.8 GiB of RAM