-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add QmlObject for future simplification of "private" properties #1921
base: main
Are you sure you want to change the base?
Conversation
The following is now possible:
|
If there are any suggestions for a better name for "QmlObject" now's a good time to change it! |
f0e1040
to
a46b54f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable enough :)
I think I hear the Minions when reading those |
d26a50c
to
92ee878
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commented about some minor things but LGTM
92ee878
to
ea102cf
Compare
You are only allowed to use this class if you can say "Batooay to quer da ads a banana" beforehand. |
Fixes #1920