ahiemstra added inline comments.

INLINE COMMENTS

> mart wrote in pagerouter.h:84
> how many routes an app would normally be going to have?
> 
> this seems really a thing for QQmlListProperty (which then each route must 
> become a qobject tough)
> so routes: [
> Route {
> 
>   name: "home"
>   component: homeComponent
>   cache: true
>   initialProperties: {"name": "value,...}
> 
> },
> Route {}
> ]
> 
> which kindof becomes similar to pagepoolaction (which i still think they can 
> be merged)

This is exactly what I was thinking. And if you mark it as default property you 
wouldn't even need to do the "routes: []" awkwardness.

> mart wrote in pagerouter.h:308
> quite ugly name..
> maybe isRouteActive

If you have routes as objects you could have an "active" property on the route, 
removing the need for this invokable.

REPOSITORY
  R169 Kirigami

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

To: cblack, #kirigami, mart, davidedmundson
Cc: ahiemstra, davidedmundson, plasma-devel, fbampaloukas, GB_2, domson, 
dkardarakos, ngraham, apol, mart

Reply via email to