Skip to content
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

Raise Publisher UI to front when already opened #924

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Sep 27, 2024

Changelog Description

Make Publisher UI raised to the front when clicking AYON > Create… or AYON > Publish... in host integrations if it was already opened.

Additional info

Especially useful for integrations that do not have Qt UI to parent to and the publisher UI sometimes going behind the DCC itself, etc.

Testing notes:

  1. Publisher UI should raise to the front when currently behind another window (or behind the DCC window)
  2. Publisher UI should still work across different integrations.

…` or `AYON > Publish...` in host integrations if it was already opened.
@BigRoy BigRoy added community Issues and PRs coming from the community members type: enhancement Improvement of existing functionality or minor addition labels Sep 27, 2024
@BigRoy BigRoy self-assigned this Sep 27, 2024
@ynbot ynbot added the size/XS label Sep 27, 2024
@@ -252,6 +252,9 @@ def show_publisher_tool(self, parent=None, controller=None, tab=None):
if tab:
window.set_current_tab(tab)
window.make_sure_is_visible()
window.raise_()
Copy link
Member

@iLLiCiTiT iLLiCiTiT Sep 30, 2024

Choose a reason for hiding this comment

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

So instead of adding it to make_sure_is_visible, you've added it here? BTW where it is not showed on calling make_sure_is_visible, because it works for me?

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.

3 participants