<div dir="ltr"><div><div>RefreshableScrollView<br><br>who sets requestingRefresh to false again? the app when it finishes refreshing?<br>If so the property is more "refreshing" than "requestingRefresh"<br><br>API docs mising<br><br>Binding{} is better than 2 Connections to implement what effectively is a binding<br>(and you have a bug here I'm sure, root.flickableItem.bottomMargin is set to different things depending on whether the view or the content changed last)<br><br>------------<br><br>ListItem<br><br>Given it doesn't contain a label AbstractListItem or BaseListItem would be a better name<br><br>A subclass which then has the icon + text would be a good idea.<br><br>Docs for delegate for a section talks about showing a title. It doesn't.<br><br>Typo: otems -> items<br><br>    height: paddingItem.childrenRect.height + Units.smallSpacing*2<br>is automatically bound to implicitHeight. No point rebinding it to the same thing manually<br><br>What's the purpose of the checked state that only changes the UI layer?<br>Do we want to make it checkable like a button?<br><br>Need to change the UI if we're enabled or not (grey out or something)<br><br>------------<br><br>ListItemWithActions<br><br>docs for actions looks like you copied and pasted it from Page.<br><br>Rather than doing JS searching for QAction and QQC Action look into qmlRegisterExtendedType for QAction to make the properties match lower in the stack<br><br>-----------------------<br><br>Heading<br><br>I don't really want this in here *and* in Plasma extras long term<br><br></div><div>and the doc formatting is randomly different.<br></div><div><br>---------------------<br>​<br></div>General: <br></div> - accessibility needs doing<br></div>