| Summary: | GridView scrolls up when I open/close a "Get hot new <thing>" window | ||
|---|---|---|---|
| Product: | [Frameworks and Libraries] frameworks-knewstuff | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | Dan Leinir Turthra Jensen <admin> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | alexander.lohnau, kdelibs-bugs, nate |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/frameworks/knewstuff/commit/1ae7749e7fdb4c981436ae7d142700ac93ef3238 | Version Fixed In: | 5.80 |
| Attachments: | screen recording | ||
This is an artefact of how the KCM is implemented (they reload everything rather than change the model in-place). Basically this is a bug in each of the KCMs, not in KNewStuff... which also means i'm not entirely sure where to point this, because this is all in widely different products... Well that's weird. Can confirm. (In reply to Nate Graham from comment #2) > Well that's weird. Can confirm. Not really weird - i explained above why it happens (and also why it, somewhat annoyingly, is both not a knewstuff bug, and also not at all easy to reassign, because it'd have to be assigned to a whole pile of products which all have the problem, because they all just reload their data when the kns dialog is closed rather than changing the model in-place) So every implementation will need to be changed? i'm afraid so... Basically, KNS sends out entry changed notifications for each changed entry, but the way most of these places handles that is to just reload their entire model, rather than identify what changed and only change that item in their model... Because, i guess, it's just easier and nobody posted a bug like this before ;) Could you follow up on that and submit MRs to fix them? (In reply to Nate Graham from comment #6) > Could you follow up on that and submit MRs to fix them? Hmm... This feels like something which might just be a good JJ type opportunity... There's a bunch of places it needs doing, but they're all self-contained, so... maybe a chance to grab some people who aren't super involved? :) A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knewstuff/-/merge_requests/105 Git commit 1ae7749e7fdb4c981436ae7d142700ac93ef3238 by Alexander Lohnau. Committed on 25/02/2021 at 15:14. Pushed by alex into branch 'master'. qtquickengine: Do not emit entriesChanged signal when we clear an empty list This is called whenever we open the dialog, but we do not want to emit the signal unless we actually clear this list. Otherwise the KCMs would reload their data and scroll back to the top. Also we do not want the changed signal to be emitted for intermediate states, this will only lead to noise on the KCM side and we still emit the signal when we have the entry updated/installed. FIXED-IN: 5.80 If we never want to scroll to the top we have to implement this on the side of the individual KCMs. M +10 -2 src/qtquick/quickengine.cpp https://invent.kde.org/frameworks/knewstuff/commit/1ae7749e7fdb4c981436ae7d142700ac93ef3238 Git commit 6359240eace78eb593968421829fbdea4148a6a4 by Alexander Lohnau. Committed on 09/04/2021 at 06:16. Pushed by alex into branch 'master'. ksplash kcm: Only update necessary parts on KNewStuff events M +1 -0 kcms/ksplash/CMakeLists.txt M +38 -14 kcms/ksplash/kcm.cpp M +5 -3 kcms/ksplash/kcm.h M +7 -2 kcms/ksplash/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-desktop/commit/6359240eace78eb593968421829fbdea4148a6a4 |
Created attachment 134821 [details] screen recording Watch the attached screenshot recording please. GridView scrolls up when I open "Get new wallpapers" or "Get new color schemes" windows and when I close "Get new window decorations" window. EXPECTED RESULT GridView should not scroll when we open/close a "Get hot new <thing" window. SOFTWARE/OS VERSIONS Operating System: KDE neon Unstable Edition KDE Plasma Version: 5.20.80 KDE Frameworks Version: 5.79.0 Qt Version: 5.15.2