latest Kirigami master breaks header size assignments

Dirk Hohndel dirk at hohndel.org
Thu Apr 21 18:37:25 UTC 2016


On Thu, Apr 21, 2016 at 08:09:15PM +0200, Marco Martin wrote:
> On Thursday 21 April 2016 10:09:38 Dirk Hohndel wrote:
> > Currently we do this:
> > 
> > Kirigami.ApplicationWindow {
> > 	id: rootItem
> > 	title: qsTr("Subsurface-mobile")
> > 
> > 	header.minimumHeight: 0
> > 	header.preferredHeight: Kirigami.Units.gridUnit * (Qt.platform.os == 
> "ios"
> > ? 2 : 1) header.maximumHeight: Kirigami.Units.gridUnit * 2 property bool
> > fullscreen: true
> > 
> > But that fails with
> > qrc:///qml/main.qml:15 Invalid grouped property access
> > (where line 15 is the header.minimumHeight: 0 assignement)
> 
> i reverted commit f11259c793f36a6a09395 that unfortunately means is again not 
> possible to assign a custom applicationheader in an application
> with that commit the only way possible to set custom heights would have been
> 
>  Kirigami.ApplicationWindow {
> 
>   header: ApplicationHeader {
> 		minimumHeight: 0
> 		...
>   }
> }

I could have lived with that... or are there other downsides I'm not seeing?

/D



More information about the Plasma-devel mailing list