Bug 414998 - Kirigami about dialog is too small
Summary: Kirigami about dialog is too small
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.64.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: Not decided
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-12-09 20:09 UTC by Matthijs Tijink
Modified: 2022-05-05 19:33 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Screenshot showing the problem (138.23 KB, image/png)
2019-12-09 20:09 UTC, Matthijs Tijink 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Matthijs Tijink 2019-12-09 20:09:53 UTC 
Created attachment 124409 [details]
Screenshot showing the problem

SUMMARY
Kirigami's About dialog/screen won't fully show the title. Probably related is that it's not correctly centered.

STEPS TO REPRODUCE
1. Open the KDE Connect SMS app
2. Press "About"

OBSERVED RESULT
The text "SMS Instant Messaging 1.4.0" only shows "SMS Instant Messagin" (without ellipsis, just falling off screen).

EXPECTED RESULT
The text should show fully, be properly centered, and either wrap or show ellipsis.
Or the about screen should cover the whole application, like it does when the window is smaller. (It still doesn't properly center the text in that case though)

SOFTWARE/OS VERSIONS

Kirigami version is actually 5.64.1 (not 5.64.0).
Comment 1 Nate Graham 2019-12-13 11:57:53 UTC 
Will fix.
Comment 2 Nate Graham 2019-12-13 13:45:46 UTC 
Bug is somewhere in here: https://cgit.kde.org/kirigami.git/tree/src/controls/AboutPage.qml#n125

Doing the obvious thing and adding an elide property isn't working; likely the layout thinks it's bigger than it is or something.
Comment 3 Felipe Kinoshita 2022-05-05 18:08:52 UTC 
This won't happen anymore since we (almost) always open the about page in a new layer, can we close this?
Comment 4 Matthijs Tijink 2022-05-05 19:33:00 UTC 
Yes, I don't see the issue any more, thanks for fixing!