Bug 457650 - Resizing Discover window from narrow to normal makes sidebar options go in the wrong place
Summary: Resizing Discover window from narrow to normal makes sidebar options go in th...
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: master
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-08-09 03:32 UTC by guimarcalsilva
Modified: 2022-09-16 13:11 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.26
Sentry Crash Report:


Attachments
Entries at the bottom are too high up (77.65 KB, image/png)
2022-08-09 03:32 UTC, guimarcalsilva
Details

Note You need to log in before you can comment on or make changes to this bug.
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