Skip to content

Conversation

@sandboxcoder
Copy link
Contributor

@sandboxcoder sandboxcoder commented Jan 14, 2026

Description

Enable "Rescale Output" dropdown on MacOS. Related PR where I enabled the checkbox but now we're revealing the dropdown as well.

image

Motivation and Context

Restoring legacy feature from 1.19.6 where you could rescale output

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@sandboxcoder sandboxcoder marked this pull request as ready for review January 14, 2026 23:46
Copy link
Contributor

@mhoyer-streamlabs mhoyer-streamlabs left a comment

Choose a reason for hiding this comment

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

Approving but curious about Mac always being visible but only for a certain encoder on Windows, why is that?

@sandboxcoder
Copy link
Contributor Author

sandboxcoder commented Jan 15, 2026

Approving but curious about Mac always being visible but only for a certain encoder on Windows, why is that?

Honestly, I think the logic is inverted the more I think about it?

git blame says it got changed here.
image

Maybe what we want here is this? Will be cleaner fix. This bug was only reported on Apple mac during the full run which was reason I went with prev fix. But open to change it to this if @aleksandr-voitenko concurs?

rescale.visible = (encoderCurrentValue != ENCODER_NVENC_H264_TEX);
rescaleRes.visible = (encoderCurrentValue != ENCODER_NVENC_H264_TEX);

[edit] This issue was discussed in slack. Went with this solution and updated PR

* I think this logic was inverted? Updating
to allow all encoders thru EXCEPT the deprecated
one
@sandboxcoder sandboxcoder force-pushed the rno/mac-rescaleoutput branch from a3f1095 to bf1beea Compare January 15, 2026 18:23
@sandboxcoder sandboxcoder changed the title settings: enable "rescale output" dropdown on macOS settings: enable "rescale output" dropdown on Windows/macOS Jan 15, 2026
@sandboxcoder sandboxcoder merged commit 9d4a690 into staging Jan 15, 2026
27 of 28 checks passed
@sandboxcoder sandboxcoder deleted the rno/mac-rescaleoutput branch January 15, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants