| Summary: | Sometimes an arrow appears before screenshot loading is completed | ||
|---|---|---|---|
| Product: | [Applications] Discover | Reporter: | Patrick Silva <bugseforuns> |
| Component: | discover | Assignee: | Aleix Pol <aleixpol> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate |
| Priority: | NOR | ||
| Version: | 5.12.2 | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/discover/a0faaed8bfbb6632314e8300886bbba4afd126cc | Version Fixed In: | 5.12.3 |
| Attachments: |
demonstration
screenshot on neon dev unstable |
||
Unrelatedly, what happens if you click on the "Sort" button that's visible in your screenshot? Something, or nothing? Hi Nate A menu with sorting options is opened. Git commit 76b90f4069c8166bf037536ae9a98f77bee8ac4c by Aleix Pol. Committed on 27/02/2018 at 13:13. Pushed by apol into branch 'Plasma/5.12'. Show a busy indicator when screenshots are fetching M +7 -1 discover/qml/ApplicationScreenshots.qml https://commits.kde.org/discover/76b90f4069c8166bf037536ae9a98f77bee8ac4c Created attachment 111415 [details]
screenshot on neon dev unstable
Not fixed on neon dev unstable.
Well we can't have the screenshot available when you please, it needs downloading. That's why we show the busy indicator. The problem in the linked screenshot is that there's a right arrow next to the busy indicator. (In reply to Nate Graham from comment #6) > The problem in the linked screenshot is that there's a right arrow next to > the busy indicator. Exactly. Because there's more screenshots to show. How would having the screenshot downloaded have an impact on the navigation? When you press the right arrow it will also show the spinning thing for the next screenshot, and then the left and right arrows. The point is, visually, it doesn't really make sense to show the arrow when there's a spinner instead of a screenshot. why was this report marked as "later"? Still valid fro Discover 5.14.1 Git commit a0faaed8bfbb6632314e8300886bbba4afd126cc by Aleix Pol. Committed on 19/10/2018 at 18:50. Pushed by apol into branch 'Plasma/5.14'. ux: hide navigation arrows while downloading a screenshot People seem to get stressed if they can navigate while downloading a screenshot. M +3 -2 discover/qml/ApplicationScreenshots.qml https://commits.kde.org/discover/a0faaed8bfbb6632314e8300886bbba4afd126cc |
Created attachment 111031 [details] demonstration See screenshot taken on neon dev unstable. Mainly flatpak apps are affected.