Created attachment 150327 [details] Preview of the bug SUMMARY At the beginning of the sidebar there is a ListSectionHeader called "default" which is inside of a ListView in a ScrollView located at qml/ViewSelector.qml. The header is correctly hidden by the Loader at the beginning by 'active: section !== "default' check, and other section headers are visible correctly. However if you scroll far away from the element, and back to it, the header is recreated as an empty slot in the ListView incorrectly. I believe delegates can remove and recreate objects as they please, and at some point it is recreated when it is not supposed to. I was looking into this and I do not believe it to be a Loader issue, as the ListView items would appear to lose their properties even without the Loader. Not sure if this is Elisa, Kirigami or Qt issue. STEPS TO REPRODUCE 1. Enable one of the embedded categories in configuration to get a longer list to your sidebar. 2. Scroll all the way to the bottom and click an object, then scroll all the way back up. 3. You should see an empty slot where the "default" ListSectionHeader is located at. OBSERVED RESULT Empty slot where "default" ListSectionHeader is located at. EXPECTED RESULT "default" ListSectionHeader is supposed to be hidden and not reloaded. SOFTWARE/OS VERSIONS KDE Plasma Version: 5.25.80 KDE Frameworks Version: 5.96.0 Qt Version: 5.15.5
Should be a Kirigami issue or a Qt issue. Moving to Kirigami for now.
A possibly relevant merge request was started @ https://invent.kde.org/multimedia/elisa/-/merge_requests/374
Git commit 137dc97f1b6a3424546b91a85f72b130b3d6a097 by Nate Graham, on behalf of Yerrey Dev. Committed on 12/07/2022 at 22:39. Pushed by ngraham into branch 'master'. Set sidebar section Loader height to avoid inactive headings taking space Add height to the Loader based on the implicit height of the loaded `item`, in the case that the `item` is null (such as when it is not active), we set the height to 0. This fixes the issue of when you have unloaded the "default" section header, and scroll back up, there would be empty space where the header would have been. M +1 -0 src/qml/ViewSelector.qml https://invent.kde.org/multimedia/elisa/commit/137dc97f1b6a3424546b91a85f72b130b3d6a097
Git commit 33cc612b8d6d7f831a8150657554beacd27c7542 by Nate Graham, on behalf of Yerrey Dev. Committed on 12/07/2022 at 22:39. Pushed by ngraham into branch 'release/22.08'. Set sidebar section Loader height to avoid inactive headings taking space Add height to the Loader based on the implicit height of the loaded `item`, in the case that the `item` is null (such as when it is not active), we set the height to 0. This fixes the issue of when you have unloaded the "default" section header, and scroll back up, there would be empty space where the header would have been. (cherry picked from commit 137dc97f1b6a3424546b91a85f72b130b3d6a097) M +1 -0 src/qml/ViewSelector.qml https://invent.kde.org/multimedia/elisa/commit/33cc612b8d6d7f831a8150657554beacd27c7542