It's a weird thing to do, and that's causing us to hit
https://bugreports.qt.io/browse/QTBUG-75900 when it tries to lookup a
property (icon) that we don't have.
BUG: 407220
Details
Details
- Reviewers
fvogt - Group Reviewers
Plasma - Commits
- R120:a282bc9d612a: Don't assign a QtObject to a model
Opened sddm-greeter --test-mode --theme clicked button
Diff Detail
Diff Detail
- Repository
- R120 Plasma Workspace
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
To do this in the past I've used an object array which would read a bit less hacky.
userListModel: [ QtObject { /*...*/} ]Comment Actions
That would still hit the Qt crash unless I declare all used properties, which is the part I wanted to avoid.
Comment Actions
Tested on top of the 5.16 beta, works fine. Please add a comment referencing the QTBUG though, so that this doesn't get reintroduced by accident.
Comment Actions
Yes, but the workaround here has no actual downsides, and the upside is avoiding a guaranteed crash with the still latest non-development release of Qt.