Bug 407118

Summary:	Keyboard navigation broken
Product:	[Applications] dolphin	Reporter:	Christoph Feck <cfeck>
Component:	view-engine: general	Assignee:	Dolphin Bug Assignee <dolphin-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nate, zrenfire
Priority:	NOR	Keywords:	regression, usability
Version:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/dolphin/038565b62f6ca6323cae2c5a23a97d66fc3dc0cd	Version Fixed In:

Description Christoph Feck 2019-05-01 07:37:18 UTC

SUMMARY
Keyboard navigation is broken

STEPS TO REPRODUCE
1. Use "icon view" mode and navigate to "/usr/lib64/"
2. Use cursor Up/Down, PageUp/PageDown keys to navigate around
3. After a few up/down cycles Dolphin starts to act weird

OBSERVED RESULT
- You can actually move "above" the first row by using "Up" multiple times
- Pressing "PageDown" does not show scroll to the newly selected item
- Pressing "Down" at the bottom edge only scrolls down ever other time

EXPECTED RESULT
Sane and working navigation

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 5.12.3
Dolphin Version: master @ ecf1ae6d0

ADDITIONAL INFORMATION
Someone also would need to investigate if the change made it into the 19.04 branch, because that's what we release next week.

Comment 1 Christoph Feck 2019-05-01 07:41:38 UTC

Forgot to say: same issues in "details view" mode, so reported as "general".

Comment 2 Christoph Feck 2019-05-21 18:29:25 UTC

Might be caused by b7db272af2f20fbeeea6bba33f4ef89d1d735bb9

Comment 3 trmdi 2019-06-05 09:41:30 UTC

Git commit 038565b62f6ca6323cae2c5a23a97d66fc3dc0cd by Tranter Madi.
Committed on 05/06/2019 at 09:40.
Pushed by trmdi into branch 'master'.

Scroll when pressing PageUp/PageDown

Summary:
Add `PageUp/PageDown` to the `navigationPressed` variable.
Remove the `m_view->firstVisibleIndex()` and `m_view->lastVisibleIndex()` tests because sometimes it does not work correctly.

Test Plan:
Press PageUp/PageDOwn: it scrolls
Press Up/DOwn: it always scrolls

Reviewers: #dolphin, Zren, cfeck, elvisangelaccio

Reviewed By: #dolphin, elvisangelaccio

Subscribers: elvisangelaccio, kfm-devel

Tags: #dolphin

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

M  +3    -4    src/kitemviews/kitemlistcontroller.cpp

https://commits.kde.org/dolphin/038565b62f6ca6323cae2c5a23a97d66fc3dc0cd