Code review comment for lp:~michael-sheldon/webbrowser-app/file-upload

Revision history for this message
Olivier Tilloy (osomon) wrote :

135 + Loader {
136 + id: peerModelLoader
137 + active: false
138 + sourceComponent: ContentPeerModel {
139 + contentType: ContentType.All
140 + handler: ContentHandler.Source
141 + }
142 + }

Does this need to be a separate component? I don’t see it reused anywhere else, maybe the customPeerModelLoader property of the ContentPickerDialog instance could instantiate it directly?

« Back to merge proposal