Bug 457650

Summary: Resizing Discover window from narrow to normal makes sidebar options go in the wrong place
Product: [Applications] Discover Reporter: guimarcalsilva
Component: discoverAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, nate, p.r.worrall, vedantwankhade17
Priority: NOR Keywords: regression
Version First Reported In: master   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.26
Sentry Crash Report:
Attachments: Entries at the bottom are too high up

Description guimarcalsilva 2022-08-09 03:32:58 UTC
Created attachment 151181 [details]
Entries at the bottom are too high up

SUMMARY

Resizing Discover window from narrow to normal makes sidebar options go in the wrong place. Check screenshot for details.

STEPS TO REPRODUCE
1. Open Discover
2. Resize it horizontally until it switches to mobile mode
3. Resize it back to full size

OBSERVED RESULT
Bottom options in the sidebar are in the wrong place

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Kernel Version: 5.15.0-43-generic (64-bit)
Graphics Platform: Wayland
Processors: 6 × Intel® Core™ i5-9400F CPU @ 2.90GHz
Memory: 7,6 GiB of RAM
Graphics Processor: Radeon RX 570 Series
Comment 1 Paul Worrall 2022-08-09 06:59:10 UTC
Marked as a regression, it was OK in 5.25.4
Comment 2 Nate Graham 2022-08-09 19:35:28 UTC
Can reproduce.
Comment 3 Vedant 2022-09-16 09:07:33 UTC
Can I work on this if no one else is?
Doesn't look too hard to fix.

The issue is that the height of the footer doesn't go back to zero after resizing the window from narrow to wide.
Need to set the height of the footer to zero if the window is widescreen.
Comment 4 Bug Janitor Service 2022-09-16 09:38:03 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/373
Comment 5 Vedant 2022-09-16 10:44:28 UTC
Git commit fe05619b82250b4e615fc6d4bb47a6eb905146fa by Vedant Wankhade.
Committed on 16/09/2022 at 09:20.
Pushed by apol into branch 'master'.

Fixes the virtical placement of the sidebar after resizing the window from narrow to wide.

Set the height of the footer to zero after resizing the window to widescreen.

M  +1    -0    discover/qml/DiscoverWindow.qml

https://invent.kde.org/plasma/discover/commit/fe05619b82250b4e615fc6d4bb47a6eb905146fa