search

librehat / yapstocks

Yet Another Plasma Stocks Applet
GNU General Public License v3.0
22 stars 7 forks source link

Layout problems #11

Closed realGWM closed 2 years ago

realGWM commented 2 years ago

Hey, thanks for your work on this. Just wanted to let you know (in case you are still maintaining this), that there is some kind of a layout problem (not sure if it is app's problem, or something wrong with my system, but this is the only app I have problems with). photo_2022-01-28_20-56-36 photo_2022-01-28_20-56-38

lah7 commented 2 years ago

I can confirm having the same issue. I think it started with one of KDE's updates a few weeks ago.

OS: Arch Linux Plasma: 5.23.5 KDE Frameworks: 5.90.0 Qt Version: 5.15.2 X11 / NVIDIA

MrLavender commented 2 years ago

Same here. It seems to be not including headerRow in layout calculations. E.g. a hack fix for mainView is

        PlasmaComponents3.ScrollView {
            anchors.fill: parent
+           anchors.topMargin: headerRow.height
            ListView {
                id: view

Operating System: Manjaro Linux KDE Plasma Version: 5.23.5 KDE Frameworks Version: 5.90.0 Qt Version: 5.15.2 Graphics Platform: X11

librehat commented 2 years ago

Hi all, I don't use Linux desktop at the moment. If you have a fix, I'm happy to accept the PR.

librehat commented 2 years ago

Back to using Linux as my daily driver, will fix this