qgroundcontrol icon indicating copy to clipboard operation
qgroundcontrol copied to clipboard

Add QmlObjectListItem

Open HTRamsey opened this issue 1 year ago • 3 comments

Provides the required functions to be used in a QmlObjectListModel. This cuts a bit down on repeated code and makes it more clear where these classes are used. @DonLakeFlyer This is what I was talking about in my reply to the other PR.

HTRamsey avatar May 05 '24 21:05 HTRamsey

Yeah, makes sense

DonLakeFlyer avatar May 06 '24 17:05 DonLakeFlyer

I think this is worthwhile, but it needs a thorough double checking. Also, I don't know if QmlObjectListModel should be changed to specifically hold QmlObjectListItems or not.

HTRamsey avatar May 06 '24 17:05 HTRamsey

Also, I don't know if QmlObjectListModel should be changed to specifically hold QmlObjectListItems or not.

50/50. A decent amount of usage of QmlObjectListModel doesn't use dirty bits. Maybe the QmlObjectListItem thing should just be something more generic to indicate a generic QObject with a dirty bit. Not sure switching the ones that don't use a dirty bit adds value.

DonLakeFlyer avatar May 07 '24 19:05 DonLakeFlyer