Bug 335675

Summary: gwenview sometimes shows "0%" zoom when cycling through images after manually setting zoom
Product: [Applications] gwenview Reporter: Martin Walch <walch.martin>
Component: generalAssignee: Gwenview Bugs <gwenview-bugs-null>
Status: RESOLVED DUPLICATE    
Severity: normal CC: j.zaitseff, myriam
Priority: NOR    
Version: 4.13.1   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: screenshot of gwenview showing the problem
Fix for bug 335675

Description Martin Walch 2014-06-02 01:36:46 UTC
Created attachment 86959 [details]
screenshot of gwenview showing the problem

instead of the actual zoom level, gwenview sometimes just shows "0%" although the image is correctly shown

to reproduce:
1. run gwenview and open a folder with at least two images
2. open the first image by clicking on it
3. change zoom level to 200%
4. step to the next image
5. step back to return to the first image

after those steps gwenview has remembered to show the first image at 200% and displays it accordingly, but says that the zoom level is 0%
Comment 1 John Zaitseff 2014-07-09 07:38:59 UTC
The reason this happens is that q->zoomChanged() is not called from within DocumentViewPrivate::setCurrentAdapter().  I am attaching a fix for this bug (as run against the current Git tree).
Comment 2 John Zaitseff 2014-07-09 07:39:43 UTC
Created attachment 87656 [details]
Fix for bug 335675
Comment 3 Christoph Feck 2014-07-20 19:51:14 UTC
John, thanks for the investigation and patch! I suggest to create a review request at https://git.reviewboard.kde.org/

Please replace tabs with spaces before posting it there.
Comment 4 Christoph Feck 2014-08-16 14:32:21 UTC
John, any success with comment #3?
Comment 5 Christoph Feck 2014-08-16 14:36:42 UTC
Ignore my last comment, this is supposed to be fixed with your other patch.

*** This bug has been marked as a duplicate of bug 337262 ***