Bug 443200

Summary: App does not fit for smaller screen.
Product: [Applications] KDE Itinerary Reporter: Gerion <gerion.entrup>
Component: generalAssignee: Volker Krause <vkrause>
Status: REOPENED ---    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Android 9.x   
Latest Commit: Version Fixed/Implemented In: 21.08.3
Sentry Crash Report:
Attachments: Screenshot_20211001-160555_KDE_Itinerary.png
Screenshot_20211001-160726_KDE_Itinerary.png
Screenshot_20211008-200659_KDE_Itinerary.png

Description Gerion 2021-10-01 14:25:52 UTC
Created attachment 142067 [details]
Screenshot_20211001-160555_KDE_Itinerary.png

SUMMARY

KDE Itinerary (with German translation) does not fit on my screen. Furthermore, I have no possibility to scroll horizontally, which makes it impossible to read the text.

STEPS TO REPRODUCE
1. Use a smartphone with a small screen.
2. Open KDE Itinerary.

OBSERVED RESULT
Please see the attached screenshots.

EXPECTED RESULT

The text fits onto the screen or is scrollable at least.

SOFTWARE/OS VERSIONS
KDE Itinerary: 21.08.1

ADDITIONAL INFORMATION

I'm using a Samsung Galaxy S4 mini (4,3", 540 x 960 px, 256 ppi).
Comment 1 Gerion 2021-10-01 14:26:11 UTC
Created attachment 142068 [details]
Screenshot_20211001-160726_KDE_Itinerary.png
Comment 2 Volker Krause 2021-10-01 16:06:39 UTC
Git commit d07f508ff39129e4e5d16f42fe63949c9d7f483f by Volker Krause.
Committed on 01/10/2021 at 16:06.
Pushed by vkrause into branch 'release/21.08'.

Also word-wrap welcome page headers

They can get rather wide in some translations.
FIXED-IN: 21.08.2

M  +6    -0    src/app/WelcomePage.qml

https://invent.kde.org/pim/itinerary/commit/d07f508ff39129e4e5d16f42fe63949c9d7f483f
Comment 3 Gerion 2021-10-02 11:44:57 UTC
Thanks. Does this fix the styling issues in the second screenshot as well?

The menu line is incomplete as well as some text and most buttons in the settings.
Comment 4 Volker Krause 2021-10-02 12:04:16 UTC
Oh, sorry, I totally missed there is a second screenshot, reopening.
Comment 5 Volker Krause 2021-10-06 15:21:28 UTC
Git commit d5099a87943e32ed00ebe91d0153c484b295a6f2 by Volker Krause.
Committed on 06/10/2021 at 15:19.
Pushed by vkrause into branch 'release/21.08'.

Don't let checkbox labels force the minimum page width
FIXED-IN: 21.08.3

M  +7    -0    src/app/SettingsPage.qml

https://invent.kde.org/pim/itinerary/commit/d5099a87943e32ed00ebe91d0153c484b295a6f2
Comment 6 Gerion 2021-10-08 18:11:05 UTC
Created attachment 142259 [details]
Screenshot_20211008-200659_KDE_Itinerary.png

Does this fix the status bar on top, too? For example the "Willkom(men)" in the second screenshot is cutted. Here is another example, where the status bar is cropped.

Also, the buttons in this screenshot overlaying each other and have different sizes. I guess that this looks way better on a wider screen.
Comment 7 Volker Krause 2021-10-13 15:26:23 UTC
There are a number of Kirigami fixes coming in 5.87 (most notably https://invent.kde.org/frameworks/kirigami/-/issues/25) which takes care of some of this. The header not being scrolled to the right remains though, and happens with all screen sizes.
Comment 8 Volker Krause 2021-10-13 18:53:55 UTC
https://invent.kde.org/frameworks/kirigami/-/merge_requests/373 fixes the header issue.
Comment 9 Gerion 2021-10-13 22:17:23 UTC
Nice, looks really well.

I will reopen this ticket to track the progress until the changes are merged. 21.08.02 has arrived in stable with a now nicely formatted help message, thanks!