D25067: Fix the header layouts for EntryDetails and Page components
Nathaniel Graham
noreply at phabricator.kde.org
Wed Oct 30 15:53:58 GMT 2019
ngraham requested changes to this revision.
ngraham added a comment.
This revision now requires changes to proceed.
Close, but not quite:
F7695250: Screenshot_20191030_095130.png <https://phabricator.kde.org/F7695250>
F7695252: Screenshot_20191030_095200.png <https://phabricator.kde.org/F7695252>
INLINE COMMENTS
> EntryDetails.qml:95
> titleDelegate: QtLayouts.RowLayout {
> + QtLayouts.Layout.fillWidth: true
> implicitHeight: title.height
I don't think this works, since it's not inside another layout. Instead maybe `anchors.fill: parent` would work better?
> Page.qml:98
> titleDelegate: QtLayouts.RowLayout {
> + QtLayouts.Layout.fillWidth: true
> Kirigami.Heading {
ditto
REPOSITORY
R304 KNewStuff
REVISION DETAIL
https://phabricator.kde.org/D25067
To: leinir, ngraham, #knewstuff, #frameworks, #plasma
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191030/39493642/attachment.html>
More information about the Kde-frameworks-devel
mailing list