Code review comment for lp:~gcollura/ubuntu-ui-toolkit/fix-1341814-and-1400297

Tim Peeters (tpeeters) wrote :

21 + property Item __oldContents: null

Instead of that, I propose to introduce:

QtObject {
  id: internal
  property Item oldContents: null
}

and then use internal.oldContents instead of __oldContents in onContentsChanged. That way you really prevent developers using PageHeadStyle from accessing __oldContents.

« Back to merge proposal