Bug 384351

Summary:	Make Update All button more visible/reachable after we scroll down the packages list
Product:	[Applications] Discover	Reporter:	soee <kontakt>
Component:	discover	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	bugseforuns, nate
Priority:	NOR	Keywords:	usability
Version:	5.10.5
Target Milestone:	---
Platform:	Neon
OS:	Linux
Latest Commit:	https://commits.kde.org/discover/91fa023160a603acc466fac868643f89926ae1e6	Version Fixed In:

Description soee 2017-09-04 13:34:40 UTC

If we have long list with packages to be updated often we scroll down to check what will be updated. From the UI/UX point of view we have a small issue here, where Update All button will be hidden when we scroll down the list. We have to scroll to top again to reach the button again. 

- maybe this top bar with this button should be sticky above the list and always visible
- maybe we could should the same button under the list
- maybe we should add sctoll to top button at the bottom

For me first option would be most efficient. Share your opinions :-)

Comment 1 Nate Graham 2017-10-19 21:47:30 UTC

+1 for making the toolbar sticky rather than inside the scroll view.

Comment 2 Nate Graham 2017-10-20 03:34:59 UTC

If we implement my suggestion for a permanent toolbar in https://bugs.kde.org/show_bug.cgi?id=385973, it will be a simple matter to put the Update All button in that toolbar, which will solve this issue.

Comment 3 Nate Graham 2017-12-12 20:11:53 UTC

I'm working on a patch that fixes this.

Comment 4 Aleix Pol 2017-12-12 23:02:57 UTC

I'd prefer a design that fixes this that we can discuss than a patch.

Comment 5 Nate Graham 2017-12-12 23:06:15 UTC

I'm not going to be wedded to my patch, but I wanted to show it in patch form rather than just an un-fleshed-out idea. In essence I'm proposing to turn the banner header into a standard-ish toolbar on top that has maximumSize: minimumSize so it doesn't resize, and then that toolbar holds everything that curently disappears when you scroll down: the update count label and the "Update all" button. I can't post a screenshot now but I can a little later.

Comment 6 Aleix Pol 2018-01-08 17:21:15 UTC

Git commit 91fa023160a603acc466fac868643f89926ae1e6 by Aleix Pol.
Committed on 08/01/2018 at 17:17.
Pushed by apol into branch 'master'.

Improve display of the updates page header

The bug was already solved, but update items would get on top of it
sometimes.
Make the header a page header rather than a listview's.

M  +55   -58   discover/qml/UpdatesPage.qml

https://commits.kde.org/discover/91fa023160a603acc466fac868643f89926ae1e6