Bug 321091

Summary:	can't use right/left arrows after cropping
Product:	[Applications] gwenview	Reporter:	daramas444
Component:	general	Assignee:	Gwenview Bugs <gwenview-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	benni, bugseforuns, myriam, nate
Priority:	NOR	Keywords:	usability
Version First Reported In:	4.10.4
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	https://commits.kde.org/gwenview/7369f4719529b7d4dd5e9ee55e22e447383ef6bb	Version Fixed/Implemented In:
Sentry Crash Report:

Description daramas444 2013-06-13 08:32:46 UTC

I have a bunch of images in a folder. I open one and can see the others by pressing on right or left arrow on the keyboard.

If I crop an image, pressing on right or left arrow (keyboard) doesn't have any effect any more. Clicking on the icons previous/next does have an effect. If I resize an image after that, pressing on right or left arrow (keyboard) has an effect again.

Reproducible: Always

Steps to Reproduce:
1. open an image in a folder containing many.
2. crop an image
3. try to use right or left arrow (keyboard), nothing happens
Actual Results:  
can't see the next or previous image using right or left arrow (keyboard)

Expected Results:  
I can see the next or previous image using right or left arrow (keyboard)

Comment 1 Nate Graham 2017-09-09 21:08:18 UTC

Confirmed.

Comment 2 Peter Mühlenpfordt 2018-04-04 06:42:05 UTC

Git commit 7369f4719529b7d4dd5e9ee55e22e447383ef6bb by Peter Mühlenpfordt.
Committed on 04/04/2018 at 06:33.
Pushed by muhlenpfordt into branch 'Applications/18.04'.

Set focus back to image view after leaving edit tool

Summary:
If you click on the toolbar of an image edit tool (crop, redeye)
the focus is lost after leaving the tool and you cannot switch images
with cursor keys.
This patch resets the focus to image view after leaving the edit tool.

Test Plan:
* Open image in Gwenview View Mode (normal or fullscreen)
* Start crop tool ({key Shift C})
* Click on {nav Crop} or {nav Cancel}
* Switching images with cursor keys should work as usual

Reviewers: #gwenview, rkflx

Reviewed By: #gwenview, rkflx

Subscribers: rkflx, huoni

Differential Revision: https://phabricator.kde.org/D11899

M  +10   -0    app/viewmainpage.cpp
M  +3    -0    app/viewmainpage.h

https://commits.kde.org/gwenview/7369f4719529b7d4dd5e9ee55e22e447383ef6bb