Bug 466815

Summary: Broken "Content Rating" layout
Product: [Applications] Discover Reporter: Mingcong Bai <jeffbai>
Component: discoverAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, guimarcalsilva, me, nate
Priority: NOR    
Version First Reported In: 5.27.2   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/discover/commit/f780c6ceaf3da7e3610d384088504364118b2f82 Version Fixed/Implemented In: 5.27.5
Sentry Crash Report:
Attachments: Incorrect text layout in "Content Rating" pop-up.

Description Mingcong Bai 2023-03-04 08:15:10 UTC 
SUMMARY

The "Content Rating" layout for some applications fails to display the full lines, as the floating window is apparently not adaptive to the content (or the content does not line-wrap based on window width). See screenshot.

STEPS TO REPRODUCE

1. Open Discover.
2. Look for Telegram Desktop from FlatHub.
3. Click on "See details..." under "Content Rating."

OBSERVED RESULT

Text in the "Content Rating" pop-up window is cut short by the window.

EXPECTED RESULT

Text in the "Content Rating" pop-up should line-wrap with the width of the window.

SOFTWARE/OS VERSIONS

Operating System: AOSC OS 10.1.0
KDE Plasma Version: 5.27.2
KDE Frameworks Version: 5.103.0
Qt Version: 5.15.5
Kernel Version: 6.2.1-aosc-main (64-bit)
Graphics Platform: X11
Processors: 8 × 11th Gen Intel® Core™ i7-1195G7 @ 2.90GHz
Memory: 15.3 GiB of RAM
Graphics Processor: Mesa Intel® Xe Graphics
Manufacturer: Panasonic Corporation
Product Name: CFSV2-1
System Version: 001
Comment 1 Mingcong Bai 2023-03-04 08:16:05 UTC 
Created attachment 156988 [details]
Incorrect text layout in "Content Rating" pop-up.
Comment 2 Nate Graham 2023-03-06 23:00:10 UTC 
Ah right, this. It was a Kirigami bug, right Ivan? Do you recall?
Comment 3 ratijas 2023-03-13 15:17:16 UTC 
Yes, it's one of the cases where Kirigami Dialogs incorrectly follow implicit size of their content.

See also: unnecessary horizontal scrolling: https://invent.kde.org/frameworks/kirigami/-/merge_requests/966
Comment 4 guimarcalsilva 2023-03-25 00:49:55 UTC 
*** Bug 465071 has been marked as a duplicate of this bug. ***
Comment 5 Bug Janitor Service 2023-04-10 17:43:37 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/532
Comment 6 Ismael Asensio 2023-04-11 15:43:40 UTC 
Git commit f0620e01f04b9307f8a9fda8e5fff27ce498ecfc by Ismael Asensio.
Committed on 11/04/2023 at 15:36.
Pushed by iasensio into branch 'master'.

ApplicationPage: Fix layout for content rating dialog

Add a larger preferred width (similar to the software risks one)
and wrap text.
FIXED-IN: 5.27.5

M  +2    -0    discover/qml/ApplicationPage.qml

https://invent.kde.org/plasma/discover/commit/f0620e01f04b9307f8a9fda8e5fff27ce498ecfc
Comment 7 Ismael Asensio 2023-04-11 15:44:16 UTC 
Git commit f780c6ceaf3da7e3610d384088504364118b2f82 by Ismael Asensio.
Committed on 11/04/2023 at 15:44.
Pushed by iasensio into branch 'Plasma/5.27'.

ApplicationPage: Fix layout for content rating dialog

Add a larger preferred width (similar to the software risks one)
and wrap text.
FIXED-IN: 5.27.5


(cherry picked from commit f0620e01f04b9307f8a9fda8e5fff27ce498ecfc)

M  +2    -0    discover/qml/ApplicationPage.qml

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