D22868: Revamp Kirigami.AboutPage

Eike Hein noreply at phabricator.kde.org
Thu Aug 1 02:03:50 BST 2019


hein created this revision.
hein added reviewers: Frameworks, VDG, mart, apol.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
hein requested review of this revision.

REVISION SUMMARY
  - Use a FormLayout to work nicely in both portrait and landscape orientations
  - Don't use an AbstractCard for the person delegate, it's very jarring in this design
  - Fix app description not having word wrap (breaks on portrait)
  - Improve inter-section spacing
  - Fix empty author email addresses showing as "Name <>"
  - Fix scrollbar length if a `Kirigami.Page` doesn't set any actions (and don't load a gigantic subtree of unnecessary objects into the page)
  - If an app uses `Kirigami.AbstractApplicationWindow` and doesn't have a `pageStack`, don't have the license delegate be a `LinkButton` that operates on one and causes an error
  - Add a fallback to KAboutData::componentName for the program icon - `KAboutData::programIconName` is deprecated, there's no QML accessor for its replacement `QGuiApplication::setWindowIcon` yet, but `KAboutData::programIconName` falls back to `componentName` already (it fails because `KAboutData::programIconName` isn't a qprop, but I don't feel like adding it to KCoreAddons for a deprecated thing while `componentName` is not deprecated)

REPOSITORY
  R169 Kirigami

BRANCH
  aboutpage

REVISION DETAIL
  https://phabricator.kde.org/D22868

AFFECTED FILES
  src/controls/AboutPage.qml
  src/controls/Page.qml

To: hein, #frameworks, #vdg, mart, apol
Cc: plasma-devel, fbampaloukas, domson, dkardarakos, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190801/2b0ecf0d/attachment.html>


More information about the Plasma-devel mailing list