Bug 491821

Summary: Vendor info overlaps a horizontal line in Update page
Product: [Applications] Discover Reporter: Patrick Silva <bugseforuns>
Component: discoverAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, kdedev, nate
Priority: NOR Keywords: regression
Version: 6.1.4   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 6.2.5
Sentry Crash Report:
Attachments: screenshot
Plasma 6.2.4

Description Patrick Silva 2024-08-17 11:58:58 UTC
Created attachment 172699 [details]
screenshot

STEPS TO REPRODUCE
1. open Discover when a update from distro repos is available
2. click on the update
3. 

OBSERVED RESULT
as we can see in the attached screenshot. vendor info overlaps an horizontal line

EXPECTED RESULT
the observed result should not occur

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.1.4
KDE Frameworks Version: 6.5.0
Qt Version: 6.7.2
Graphics Platform: Wayland
Comment 1 Nate Graham 2024-08-20 16:53:27 UTC
Can't reproduce on Fedora because they don't set the vendor tag. I guess this only happens on distros that do.
Comment 2 Nate Graham 2024-08-27 23:39:13 UTC
Looks like it's a general layout regression here. Stuff that used to be arranged in rows are now in a column.
Comment 3 Nate Graham 2024-08-28 00:04:28 UTC
Ah, that was something else that's only in master (and fixed now).

How reproducible is this, Patrick? 100% of the time? Or only some of the time? Does it fix itself if you resize the window?
Comment 4 Patrick Silva 2024-08-28 00:13:44 UTC
I can reproduce by clicking on a update for the first time after opening Discover (and the bug persists after window resizing).
If I go to another page, go back to "Update" page and then click on the previously clicked update again, the "Vendor" url is missing.
Comment 5 Nate Graham 2024-11-27 20:55:11 UTC
Is it still reproducible for you on git master or 6.2? I can't seem to make it happen.
Comment 6 Patrick Silva 2024-11-27 22:14:44 UTC
Created attachment 176176 [details]
Plasma 6.2.4

The bug persists on Plasma 6.2.4.

Operating System: Arch Linux 
KDE Plasma Version: 6.2.4
KDE Frameworks Version: 6.8.0
Qt Version: 6.8.0
Graphics Platform: Wayland
Comment 7 TraceyC 2024-12-03 16:36:36 UTC
Confirmed this issue persists for me as well on 6.2.4 (Solus). I'm not able to test on git-master at the moment
Comment 8 Aleix Pol 2024-12-05 01:38:26 UTC
It does happen on master, Qt 6.8, all clean.

I'm pretty sure the size calculation of Text {} with StyleFormat has regressed lately. 
https://doc.qt.io/qt-6/qml-qtquick-text.html#textFormat-prop

Not super sure what the right way forward is then...
Comment 9 Bug Janitor Service 2024-12-06 23:36:49 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/983
Comment 10 Aleix Pol 2024-12-11 01:03:48 UTC
Git commit 3ac19d1f5f812ae8a12cd7b00626da41c0ebc578 by Aleix Pol Gonzalez, on behalf of Aleix Pol.
Committed on 11/12/2024 at 01:03.
Pushed by apol into branch 'master'.

UpdatesPage: Fix update description box overlapping with its text

Account for margins.

M  +1    -1    discover/qml/UpdatesPage.qml

https://invent.kde.org/plasma/discover/-/commit/3ac19d1f5f812ae8a12cd7b00626da41c0ebc578
Comment 11 Aleix Pol 2024-12-11 01:04:41 UTC
Git commit 453cb49cc9509aa9a00c40de04c32fae1f50cdf3 by Aleix Pol Gonzalez.
Committed on 11/12/2024 at 01:04.
Pushed by apol into branch 'Plasma/6.2'.

UpdatesPage: Fix update description box overlapping with its text

Account for margins.


(cherry picked from commit 3ac19d1f5f812ae8a12cd7b00626da41c0ebc578)

Co-authored-by: Aleix Pol <aleixpol@kde.org>

M  +1    -1    discover/qml/UpdatesPage.qml

https://invent.kde.org/plasma/discover/-/commit/453cb49cc9509aa9a00c40de04c32fae1f50cdf3