Skip to content

Conversation

@FuzzkingCool
Copy link
Contributor

@FuzzkingCool FuzzkingCool commented Oct 30, 2025

Changelog Description

Include the current application name in AYON tools Loader, Workfiles, Publisher, and Scene Inventory.

Additional Info

This pull request updates the window titles for several AYON tool windows to dynamically include the current application name, improving context for users working across multiple host applications. The changes introduce a new title property in each window class, which constructs the window title using either the AYON_APP_NAME environment variable or the value returned by get_current_host_name().

These changes are for the Loader, Workfiles, Publisher, and Scene Inventory.

Title looks like this when launched from a DCC like Harmony:
image

And in Windows 11 for example, when hovering over the Taskbar item, clearly shows the DCC this tool belongs to.
image

Window Title Improvements:

  • Added a title property to LoaderWindow, PublisherWindow, SceneInventoryWindow, and WorkfilesToolWindow classes to dynamically generate window titles with the current application name, using AYON_APP_NAME or get_current_host_name() if available. [1] [2] [3] [4]
  • Updated window initialization in all affected classes to use the new title property for setting the window title, replacing previous static titles. [1] [2] [3]

Imports and Dependencies:

  • Added imports for os and get_current_host_name in all relevant files to support dynamic title generation. [1] [2] [3] [4]

Testing Notes

  1. launch dcc app via launcher
  2. open AYON tools and check the window title name

@ynbot ynbot added the size/XS label Oct 30, 2025
@MustafaJafar MustafaJafar added community Issues and PRs coming from the community members type: enhancement Improvement of existing functionality or minor addition labels Oct 30, 2025
Copy link
Member

@MustafaJafar MustafaJafar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works. Thanks.
Merging decision should be up to @iLLiCiTiT
image
image

Copy link
Collaborator

@BigRoy BigRoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it visually. Code seems ok, but I think @iLLiCiTiT will be the more critical one in this area, so pending his code review.

Comment on lines +134 to +135
@property
def title(self):
Copy link
Member

@iLLiCiTiT iLLiCiTiT Oct 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be method, not property, moved below (after public methods). And the app name should be defined in controller, I really do try a lot to not use anything host related in UI code.

Suggested change
@property
def title(self):
def _get_title(self):

Copy link
Member

@iLLiCiTiT iLLiCiTiT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please move pipeline logic to backend (controller).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Issues and PRs coming from the community members size/XS type: enhancement Improvement of existing functionality or minor addition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants