bradenjennings

Results 22 comments of bradenjennings

I quickly prototyped something in the following PR. This is without any guidance. So its a very rudimentary solution. This was done before you posted your code example above [Comparing...

I had a look at your code @antirotor. and I'm not sure how I would implement something like this especially in regards to signals. I'm not assigned to the ticket...

Sorry the PR was named incorrectly, I meant this one. Same code. https://github.com/ynput/ayon-core/compare/enhancement/AY-2420_callbacks_and_groups_with_publisher_attributes?expand=1

Revert to default menu options now appears for Bool, Int, Float, String and Enum menus. It adds itself to existing menus, instead of replacing them. ![image](https://github.com/ynput/ayon-core/assets/10794257/3e098991-b230-486b-a6fa-5e3f5e33638a)

There is a separate ticket for a reset all attributes button. So in this case you wouldn't need to click every attribute directly. https://github.com/ynput/ayon-core/pull/153 What's the consensus should we keep...

I find the Houdini context menu option to be a clean non distacting implementation.

Here is an example image of "Reset attributes to default" button added ![image](https://github.com/ynput/ayon-core/assets/10794257/f8fbea6c-6e04-46fc-8754-6294288da989)

Here is an example image of the implementation, which makes overridden attributes bold. ![image](https://github.com/ynput/ayon-core/assets/10794257/3cebfd66-6773-487c-86e2-bab28203fd65)

> With this making the label bold instead of the value (like Houdini does) I have the feeling that maybe it's not as clear, in particular because it makes the...

> * The 'nondefault' state does not trigger if you change a value from non-default, save, then reset publisher. > * The 'nondefault' state does not update when clicking between...