-
-
Notifications
You must be signed in to change notification settings - Fork 90
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
Support root property updates for JS #115
Conversation
} | ||
adoptChild(object_preview, object_root); |
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.
since child
and content
are properties, and we now update all properties, we don't need this anymore
title: "My App"; | ||
description: "My App is awesome"; | ||
vexpand: true; | ||
icon-name: "applications-science-symbolic"; |
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.
I prefer using a "standard" icon for this demo
@@ -1,47 +1,39 @@ | |||
using Gtk 4.0; | |||
using Adw 1; | |||
|
|||
Adw.ApplicationWindow window { | |||
Gtk.ApplicationWindow window { |
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.
no reason to use Adw.Window / Adw.ApplicationWindow if we don't want a freeform window
icon-name: "open-menu-symbolic"; | ||
primary: true; | ||
} | ||
title: "My App"; |
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.
updating a root property, such as title
now works as expected
primary: true; | ||
} | ||
title: "My App"; | ||
titlebar: Gtk.HeaderBar { |
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.
same for widget properties of course, try removing it for example
@lw64 could you test/review this and once satisfied—implement the same algorithm on the Vala side in this branch ? |
This comment was marked as resolved.
This comment was marked as resolved.
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.
Seems all good to me.
Fixes #60
Needs the Vala equivalent and ready to go (as far as I can tell)