Bug 359783

Upstream GridView bug, still needs fixing.

I can confirm that this bug it's still present on KDE Plasma 5.17.3 using KDE Neon.

I configured them right aligned and they start at left and then some times they move at start to the right (with animation) and other they align as right aligned but keep on the left side of the screen.

Can still reproduce with git master and Qt 5.13.1.

Eike, do you happen to have the URL of the GridView bug handy? Let's not mark this as RESOLVED UPSTREAM until we have a known Qt bug tracking it.

*** Bug 422035 has been marked as a duplicate of this bug. ***

*** Bug 417644 has been marked as a duplicate of this bug. ***

*** Bug 426043 has been marked as a duplicate of this bug. ***

Dupes piling up, setting to VHI.

*** Bug 426766 has been marked as a duplicate of this bug. ***

(In reply to Patrick Silva from comment #8)
> *** Bug 426766 has been marked as a duplicate of this bug. ***

I did a search, but could not find anything. Thanks for pointing me towards this bug report.

*** Bug 427001 has been marked as a duplicate of this bug. ***

*** Bug 429765 has been marked as a duplicate of this bug. ***

This is reproducible for me when I restart Plasma with a vertical panel, but not with a horizontal panel.

Created attachment 134655 [details]
Screenshot

(In reply to Nate Graham from comment #12)
> This is reproducible for me when I restart Plasma with a vertical panel, but
> not with a horizontal panel.

This is reproducible for me:
- with a horizontal panel
- without the need to restart Plasma
I can just drag a file to the desktop and the files are moved. See screenshot attached. The current title is therefore not accurate and that's why I'm changing it to something broader.

(In reply to Riccardo Robecchi from comment #13)
> I can just drag a file to the desktop and the files are moved. 

I obviously meant the icons, sorry for the confusion.

Created attachment 134656 [details]
Jumping icons without panels as well

(In reply to Nate Graham from comment #12)
> This is reproducible for me when I restart Plasma with a vertical panel, but
> not with a horizontal panel.

I've tested without panels, the issue still exists...

I also can reproduce on neon unstable even with horizontal panel on bottom.

Huh.

I almost published a bug report before I realized that there was already one. 

This bug happens under the same circumstances in openSUSE Tumbleweed in the latest version. However, it happens for me that this bug does not happen (it seems random).

just to log my findings, i thought that there possibly could have a resize event of the desktop view early in startup from small to big, with the layout code of folderview not correctly updating.. does not seem the case though

Created attachment 134923 [details]
vertical panel settings overlap

(In reply to Marco Martin from comment #19)
> just to log my findings, i thought that there possibly could have a resize
> event of the desktop view early in startup from small to big, with the
> layout code of folderview not correctly updating.. does not seem the case
> though

Maybe related to this issue that settings (if opened - on attahed picture)  of vertical panel if vertical panel is exists, overlap the horizontal panel as well

a likely culript is indeed struts and availablescreenrect that gets wrong

can you (or anybody that can reproduce) install the simple plasmoid https://www.pling.com/p/1472837/ and put it in the desktop? it should show some numbers (upload a screenshot then) that may shed some light if my theory is correct

(In reply to Marco Martin from comment #22)
> can you (or anybody that can reproduce) install the simple plasmoid
> https://www.pling.com/p/1472837/ and put it in the desktop? it should show
> some numbers (upload a screenshot then) that may shed some light if my
> theory is correct

Just tried it, it doesn't seem to detect any changes.

(In reply to Marco Martin from comment #22)
> can you (or anybody that can reproduce) install the simple plasmoid
> https://www.pling.com/p/1472837/ and put it in the desktop? it should show
> some numbers (upload a screenshot then) that may shed some light if my
> theory is correct

I tried as well, changed/changes 0 times

Created attachment 135018 [details]
Changes after relogin

I tried with relogin, attached screenshot

Created attachment 135019 [details]
screenshot of Containment Debug Plasmoid widget

here is my screenshot after I reproduce the bug on neon unstable by dragging an icon on desktop horizontally and restart Plasma running "plasmashell --replace".

I made an additional trial: I restarted plasma (kquitapp5 plasmashell && kstart5 plasmashell) and that spammed the widget with resizes. After that, though, reproducing the bug doesn't add anything new.

*** Bug 433696 has been marked as a duplicate of this bug. ***

Created attachment 136327 [details]
Video of the issue

It seems like moving icons to new columns, so that the number of columns changes, is what triggers the issue. See video for reference.

Reproducible on Arch Linux with kernel 5.11.2 and KDE Plasma 5.21.1, Vertical panel.

This annoying bug appears to be fixed by the change made to fix Bug 436216!

Thanks Oleg!

This bug persists on Arch Linux after update to Plasma 5.21.5 and reboot.

I'm starting to think that your computer is cursed, with all these bugs you're re-opening. It's fixed for me. :/ Maybe the full fix requires Plasma 5.22? Can you try again with Neon Unstable?

I can also reproduce on neon unstable installed on another computer.
If the bug really persists, probably another affected user will comment here later.

(In reply to Nate Graham from comment #31)
> This annoying bug appears to be fixed by the change made to fix Bug 436216!
> 
> Thanks Oleg!

I can confirm it's still present on my machines with Plasma 5.21.5, so I am marking it as "confirmed" once again (sigh...).

*** Bug 438030 has been marked as a duplicate of this bug. ***

(In reply to Patrick Silva from comment #36)
> *** Bug 438030 has been marked as a duplicate of this bug. ***

Thanks

Created attachment 138989 [details]
Backtraces

Can't reproduce with Qt 5.13.2

(In reply to Eike Hein from comment #1)
> Upstream GridView bug, still needs fixing.

It's a qt-quickcontrols (not quickcontrols2) issue.
5.12 is fine, 5.13 is broken.

Breaks since 3d44442331ba6f1ba7a5d5ad9f3834993911e288 in qtquickcontrols

That commit was in 2019 though, and the bug report was originally opened in 2016.

Either way, I guess we need a Qt bug report. If you don't know of one, would you be able to flie one?

Created attachment 139414 [details]
attachment-28640-0.html

I've created an account in Qt but being of no company it seems that I can
only search for opened bugs instead of open a new one, I couldn't find this
issue in the open bugs, but maybe there is one with technical info on it.

El mié, 16 de jun. de 2021 a la(s) 11:55, Nate Graham (
bugzilla_noreply@kde.org) escribió:

> https://bugs.kde.org/show_bug.cgi?id=359783
>
> --- Comment #42 from Nate Graham <nate@kde.org> ---
> That commit was in 2019 though, and the bug report was originally opened in
> 2016.
>
> Either way, I guess we need a Qt bug report. If you don't know of one,
> would
> you be able to flie one?
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.

(In reply to Nate Graham from comment #42)
> That commit was in 2019 though, and the bug report was originally opened in
> 2016.
You can check yourself whether reverting that commit fixes this bug or not.

> Either way, I guess we need a Qt bug report. If you don't know of one, would
> you be able to flie one?
Cannot file a bug yet, sorry but I'll do it later since I have a Qt account

I can't reliably reproduce it though. :(

(In reply to Nate Graham from comment #45)
> I can't reliably reproduce it though. :(

I'll try on my Gentoo (see 438030 for how I reproduce it)

(In reply to Oleg Solovyov from comment #46)
> (In reply to Nate Graham from comment #45)
> > I can't reliably reproduce it though. :(
> 
> I'll try on my Gentoo (see 438030 for how I reproduce it)

Just tried, 438030 is fixed.
Don't think it's a duplicate.

OK, feel free to un-dupe and mark as resolved.

(In reply to Nate Graham from comment #48)
> OK, feel free to un-dupe and mark as resolved.

Un-duped but it's not resolved unless 3d44442 is reverted.
AFAIK, KDE have the whole QT forked since 5.15 is not supported/published after Qt6 release.

*** Bug 440034 has been marked as a duplicate of this bug. ***

It seems like this bug is caused by QQC1.ScrollView. Getting rid of it or replacing it with QQC2.ScrollView sort of fixes the problem. I don't know why QQC1.ScrollView causes this bug.

With a quick-and-dirty patch that removes PlasmaExtras.ScrollArea (aka QQC1.ScrollView) https://invent.kde.org/-/snippets/1791, icons are aligned as expected.

https://invent.kde.org/plasma/plasma-desktop/-/blob/56f49e47479dc7fc040922f3062ceddb01639d02/containments/desktop/package/contents/ui/FolderView.qml#L622

(In reply to Vlad Zahorodnii from comment #51)
> With a quick-and-dirty patch that removes PlasmaExtras.ScrollArea (aka
> QQC1.ScrollView) https://invent.kde.org/-/snippets/1791, icons are aligned
> as expected.

Working! Big thank you

I can confirm this bug in Manjaro and Garuda Linux.

KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.2

The bug happens anytime a new icon is added to the desktop, or an icon is deleted or shifted. There is no need to re-login.
Moreover, if icons are sorted by type, when a new icon is added to the desktop, they become automatically unsorted.

Created attachment 140793 [details]
Recording of the screen showing the bugs

The icons realign to the right whenever the desktop is left-clicked.

(In reply to Nuno Gomes from comment #54)
> Created attachment 140793 [details]
> Recording of the screen showing the bugs
> 
> The icons realign to the right whenever the desktop is left-clicked.

You need apply Vlad's patch to fix this issue on Manjaro as well. We updated already Plasma on blackPanther OS, and patch solve this issue

(In reply to vector from comment #55)

> You need apply Vlad's patch to fix this issue on Manjaro as well. We updated
> already Plasma on blackPanther OS, and patch solve this issue

Thanks. I don't know how to do it in an effective way, I mean, without going to FolderView.qml and change every line manually. Any hint how to do it quickly and safely?

I wouldn't recommend that distros use Vlad's patch. It removes scrollability entirely, so users with tons and tons of desktop icons will lose the ability to access some of them from the desktop. It's a fairly drastic patch.

(In reply to Nate Graham from comment #57)
> I wouldn't recommend that distros use Vlad's patch. It removes scrollability
> entirely, so users with tons and tons of desktop icons will lose the ability
> to access some of them from the desktop. It's a fairly drastic patch.

Good to know. Thanks for the heads-up!

*** Bug 438030 has been marked as a duplicate of this bug. ***

*** Bug 442667 has been marked as a duplicate of this bug. ***

Plasma 5.24 beta on Arch Linux is also affected.

This has been an issue forever and I’ve been too lazy to report it. Glad it’s not only me.

Best way to reproduce: Move a file from the desktop to a new, empty COLUMN. This is important. You will notice icons immediately shift. Click the desktop to revert them back to how they were before.

*** Bug 449676 has been marked as a duplicate of this bug. ***

*** Bug 441477 has been marked as a duplicate of this bug. ***

*** Bug 450403 has been marked as a duplicate of this bug. ***

Same bug for me on Endeavour OS, plasma 5.24.2. Something about default position on the grid.

This is working perfectly for me with current git master, which includes the fix for Bug 354802 that also got backported to Plasma 5.24.5. I suspect that change might have fixed this issue too. For any of the people who have experienced this issue in the past, can you test again with either the current git master version of Plasma or else Plasma 5.24.5 once it's released in a week? Thanks!

Still happens for me but it's also in specific situations. They need to be right aligned first. Then if you add a new column of icons, nice the existing ones or resize the panel size, either with plama panels or latte ones, they move back to the left until you click on them again. This happens a lot with me. I can record a video demonstrating it if needed

Move the existing ones. Sorry, that was auto correct

Ryu, are you using Plasma from the master or Plasma/5.24 branch? Because the change hasn't made it into any released version yet.

(In reply to Nate Graham from comment #70)
> Ryu, are you using Plasma from the master or Plasma/5.24 branch? Because the
> change hasn't made it into any released version yet.

As I can't built using kdesrc-build, I am using KDE Unstable Repos available for openSUSE, so I am in master. I can't tell which commit, though. I updated to latest packages today, however.

It's been a few days, so hopefully the packages have been updated to include the changes. Can you test again?

The bug is still happening when I drag an icon on desktop horizontally.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.3
Graphics Platform: Wayland

I just updated to the latest available. I'm also having the same behavior of the icons when I move them around

If all of you who are using git master software can still reproduce the issue, it must be real and I'm just failing to reproduce it myself. Can one of you make a screen recording that shows a specific set of steps to reproduce it?

Created attachment 148544 [details]
video recorded on neon unstable

(In reply to Patrick Silva from comment #76)
> Created attachment 148544 [details]
> video recorded on neon unstable

Same behavior with me. Though I may note that it also happens on activity change. I think it has to do with how the icons are, by default, left aligned, and the right alignment configuration is not processed every time a change happens to the desktop icons. May be costy to check the configuration file every time but maybe it could be loaded into cache. It's just a guess, though. I don't know how the technical side of all this is.

In progress with https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1144!

Git commit 9035b461fef9ebd1fe2aad6726996c9e2d6e29fe by Nate Graham, on behalf of Marco Martin.
Committed on 13/10/2022 at 19:19.
Pushed by ngraham into branch 'master'.

Folderview: port to use ScrollView instead of ScrollArea

ScrollArea uses Qtquickcontrols1, so won't be able to run on qt6.
Don't use a ScrollView there as it always accepts inputs, so have to use a scrollbar directly in the flickable
FIXED-IN: 5.27

M  +9    -2    containments/desktop/package/contents/ui/FolderView.qml

https://invent.kde.org/plasma/plasma-desktop/commit/9035b461fef9ebd1fe2aad6726996c9e2d6e29fe

Finally it's fixed on neon unstable! \o/
Can the fix for this old bug be backported to 5.26 branch? It would be very nice.

(In reply to Patrick Silva from comment #80)
> Finally it's fixed on neon unstable! \o/
> Can the fix for this old bug be backported to 5.26 branch? It would be very
> nice.

No, it depends on KF changes.

Created attachment 160365 [details]
Icons before

Just reproduced on today's:
KDE Neon User Edition: Plasma 5.27.6, KF 5.108.0, Qt 5.15.10
KDE Neon Testing Edition: Plasma 5.27.6, KF 5.109.0, Qt 5.15.10

Sometimes machine should be rebooted for dozens of times to get this bug reproduced again.

Created attachment 160366 [details]
Icons after reboot

Forgot to reopen yesterday

Oleg, those screenshots depict an issue that's not the one reported here. You also indicate that it's not 100% reproducible like this issue was. Please submit a new bug report for that separate issue. Thanks.