Bug 401960

Summary:	Middle apps list jumps after selection any app from the list after installing/trying to install some app from this list
Product:	[Applications] Discover	Reporter:	Alexander Mentyu <notuxius>
Component:	discover	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	normal	CC:	nate
Priority:	NOR
Version:	5.14.4
Target Milestone:	---
Platform:	Neon
OS:	Linux
Latest Commit:	https://commits.kde.org/kirigami/a9dbeaf219362bad4003cbc7b6675e19ccd31b5f	Version Fixed In:
Attachments:	Selecting Emacs makes middle apps list jump back to Transmission app

Description Alexander Mentyu 2018-12-10 14:47:11 UTC

STEPS TO REPRODUCE
1. Select any top left page - for example 'Applications'.
2. Select any app on this page (middle apps list appears).
3. Select install or remove button of any app in the middle list.
4. Decline pass prompt, press circle cancel button or install the app.
5. Scroll the middle apps list.
6. Select any app in the list.

OBSERVED RESULT
Apps list jumps

EXPECTED RESULT
App from the list is selected

SOFTWARE/OS VERSIONS
Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.14.80
KDE Frameworks Version: 5.54.0
Qt Version: 5.11.2
Kernel Version: 4.15.0-42-generic

Comment 1 Nate Graham 2018-12-11 19:21:30 UTC

Confirmed.

Comment 2 Aleix Pol 2018-12-13 03:07:37 UTC

I really cannot reproduce, can you maybe take a video in case I didn't understand?

Comment 3 Alexander Mentyu 2018-12-13 13:11:20 UTC

Created attachment 116892 [details]
Selecting Emacs makes middle apps list jump back to Transmission app

Comment 4 Alexander Mentyu 2018-12-13 13:11:44 UTC

Added video with demo

Comment 5 Aleix Pol 2018-12-13 15:08:55 UTC

https://phabricator.kde.org/D17562

Comment 6 Aleix Pol 2018-12-13 15:36:09 UTC

Git commit a9dbeaf219362bad4003cbc7b6675e19ccd31b5f by Aleix Pol.
Committed on 13/12/2018 at 15:34.
Pushed by apol into branch 'master'.

Don't try to reposition the ScrollView when we get a faulty height

Summary: BUG: 401960

Test Plan: Tried as explained in the bug, can't see the issue anymore

Reviewers: #kirigami, mart

Reviewed By: #kirigami, mart

Subscribers: plasma-devel

Tags: #kirigami

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

M  +1    -1    src/controls/private/RefreshableScrollView.qml

https://commits.kde.org/kirigami/a9dbeaf219362bad4003cbc7b6675e19ccd31b5f