Bug 448280 - Discover needs better progress indicators
Summary: Discover needs better progress indicators
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: 5.23.4
Platform: Fedora RPMs Linux
: NOR minor
Target Milestone: ---
Assignee: Dan Leinir Turthra Jensen
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2022-01-11 19:08 UTC by P Fudd
Modified: 2022-10-13 16:08 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.26.1


Attachments
Screenshot of app after clicking 'install' and waiting a while. (218.60 KB, image/png)
2022-01-11 19:08 UTC, P Fudd
Details

Note You need to log in before you can comment on or make changes to this bug.
Description P Fudd 2022-01-11 19:08:13 UTC
Created attachment 145338 [details]
Screenshot of app after clicking 'install' and waiting a while.

SUMMARY
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***
Hi; 
In Discover, it's not obvious when large updates are being installed; after clicking, the display appears frozen.

STEPS TO REPRODUCE
1. Wait for a system update containing ~200 package updates
2. Select the update, click 'update'
3. Wait

OBSERVED RESULT
The app appears to hang for minutes.  There's an icon at the bottom that looks like it might be for cancelling the update, but it's hard to say for sure.

EXPECTED RESULT
* There should be some signs that work is being done and whether it can be safely interrupted.
* It would be best if there was a progress bar with an ETA calculation showing when the user will be needed to click on something (system reboot?) or process will finish.
* Second best is just a progress bar (calibrated by time, not 'number of packages')
* Third best is a 'busy' indicator

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora 35
(available in About System)
KDE Plasma Version: 5.23.4-1
KDE Frameworks Version: 5.89.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION
Screenshot attached.
Also filed as https://bugzilla.redhat.com/show_bug.cgi?id=2039471
Comment 1 Aleix Pol 2022-10-13 15:57:12 UTC
Git commit 8226ddf15fd7f04b18fc2d2f571215b695a9b2bb by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 13/10/2022 at 15:52.
Pushed by apol into branch 'master'.

flatpak: Address transaction progress skipping

Flatpak transaction progress skips back and forth as it is reported
according its different operations.
Report the progress according equally split among the different
operations.
Related: bug 404819, bug 434948, bug 435450

M  +10   -4    libdiscover/backends/FlatpakBackend/FlatpakTransactionThread.cpp
M  +3    -0    libdiscover/backends/FlatpakBackend/FlatpakTransactionThread.h
M  +6    -0    libdiscover/backends/FlatpakBackend/tests/FlatpakTest.cpp

https://invent.kde.org/plasma/discover/commit/8226ddf15fd7f04b18fc2d2f571215b695a9b2bb
Comment 2 Nate Graham 2022-10-13 16:08:05 UTC
Git commit 336becd3ad0f8aa5b18d3c61b38c15d81a3f3544 by Nate Graham, on behalf of Aleix Pol.
Committed on 13/10/2022 at 16:07.
Pushed by ngraham into branch 'Plasma/5.26'.

flatpak: Address transaction progress skipping

Flatpak transaction progress skips back and forth as it is reported
according its different operations.
Report the progress according equally split among the different
operations.
Related: bug 404819, bug 434948, bug 435450


(cherry picked from commit 8226ddf15fd7f04b18fc2d2f571215b695a9b2bb)

M  +10   -4    libdiscover/backends/FlatpakBackend/FlatpakTransactionThread.cpp
M  +3    -0    libdiscover/backends/FlatpakBackend/FlatpakTransactionThread.h
M  +6    -0    libdiscover/backends/FlatpakBackend/tests/FlatpakTest.cpp

https://invent.kde.org/plasma/discover/commit/336becd3ad0f8aa5b18d3c61b38c15d81a3f3544