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

Publisher: Screen capture fixes #855

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Aug 26, 2024

Changelog Description

Disable multi-screen screengrab to avoid issues related to scale ratio.

Additional info

Now each screen has own dialog window (which makes it faster BTW). With that we make sure that each screen is covered with the correct size.

Removed option to capture screen across multiple screens. That is related to issues connected to different scale ration causing output image size issues.

NOTE

Surprise for me, I don't know how to take screenshot of desktop, it only take screenshot of background and windows of the process it is running in.

Testing notes:

Some of the issues happened everywhere, but on macOs the biggest issue was that capture area was randomly moved on scaled multi-screen, so it was not possible to capture screen of some areas at all.

  • ideally at least of your screens should have some scaling
  1. Open publisher in host of your choice (ideally traypublisher and one DCC with Qt like Nuke or Maya).
  2. Create any instance.
  3. Select the instance and go to Publish tab.
  4. Hover over thumbnail widget and press "Take screenshot".
  5. It should be possible to take screenshot on any screen.
  6. The output should match the real selection.

@ynbot
Copy link
Contributor

ynbot commented Aug 26, 2024

@ynbot ynbot added size/S type: enhancement Improvement of existing functionality or minor addition labels Aug 26, 2024
@iLLiCiTiT iLLiCiTiT merged commit 90f5002 into develop Sep 3, 2024
6 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/AY-6472_Nuke-screenshot-grabbing-on-multi-monitor-setup branch September 3, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants