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

DolphinQt: Tweak the aspect ratio setting description #12609

Merged

Conversation

JosJuice
Copy link
Member

@JosJuice JosJuice commented Mar 2, 2024

With this, I intend to make it clearer that Auto, Force 4:3, Force 16:9 and Custom are really the same thing, just with the aspect ratio of the simulated TV being selected in a different way. I also extended the introduction in a way I feel will clarify things but which you are welcome to bikeshed :)

I was thinking of this during the review of PR #12438, but wanted to put it in a separate PR as to avoid blocking it on bikeshedding.

I'm a bit unsure what to do about the word "analog" in "analog TV". I felt that repeating it for each of these options would be too repetitive. I suppose there's a reason why we used the word originally, but digital TVs do give you basically the same aspect ratio for GC/Wii games as analog TVs. (Of course, whether it's 4:3-like or 16:9-like depends on what aspect ratio you set in the TV's settings, but that's the case for widescreen CRTs too.)

@AdmiralCurtiss
Copy link
Contributor

This looks good to me. @MayImilae Opinions?

Copy link
Contributor

@MayImilae MayImilae left a comment

Choose a reason for hiding this comment

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

This is a substantial improvement over the previous descriptions, imo. I'm not super happy with the "like a TV" language but that's fine.

I have a small quibble though.

"This will usually make proportions look distorted."
"<br><br><b>Custom</b>: Mimics a TV with the specified aspect ratio. "
"This is mostly intended to be used with aspect ratio cheats/mods."
"<br><br><b>Custom (Stretch)</b>: Similar to `Custom`, but stretches the picture as if to "
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not very happy with "stretches the picture as if to complete fill the TV".

How about...

Similar to 'Custom', but stretches the image to the specified aspect ratio. This will usually distort the image, and should not be used under normal circumstances.

Copy link
Member Author

@JosJuice JosJuice Mar 24, 2024

Choose a reason for hiding this comment

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

"stretches the image to the specified aspect ratio" sounds fine to me.

Do you have a preference for "picture" vs "image"? I'd be fine with either, but I think it looks better if we consistently use the same term throughout the description.

Copy link
Contributor

Choose a reason for hiding this comment

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

I personally prefer image.

It's a very subtle english language thing, probably even regional, but to me, "picture" is the absolute final result after everything including display (literally photons into eyeballs level), whereas "image" is more generally referring to the pixels being made and sent to the display. For example, "that television has a nice picture" is referring to the televisions ability to emit the signal that was sent to it, while "points at television that's a nice image" is referring to what the signal being sent to the television. "that television has a nice image" is not a thing in american english, as far as I know.

I'm not sure the origins of that, I believe it's from television nomenclature? Language is weird. ┐(´-`)┌

Copy link
Member Author

@JosJuice JosJuice Mar 24, 2024

Choose a reason for hiding this comment

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

I applied your suggestion, except I changed "distort the image" to "distort the image's proportions" to make it clearer what kind of distortion we're talking about. "Distort the image" on its own feels to me like to could be referring to anything from signal quality degradation to the image being cut off or sheared. I suppose since we're already talking about aspect ratios, the user might be able to infer what we mean, but I don't trust everyone to infer that.

Also tweaked the rest of the description to keep the terminology consistent.

With this, I intend to make it clearer that Auto, Force 4:3, Force 16:9
and Custom are really the same thing, just with the aspect ratio of the
simulated TV being selected in a different way. I also extended the
introduction in a way I feel will clarify things but which you are
welcome to bikeshed :)

I was thinking of this during the review of 41b19e2, but wanted to
put it in a separate PR as to avoid blocking it on bikeshedding.

I'm a bit unsure what to do about the word "analog" in "analog TV". I
felt that repeating it for each of these options would be too
repetitive. I suppose there's a reason why we used the word originally,
but digital TVs do give you basically the same aspect ratio for GC/Wii
games as analog TVs. (Of course, whether it's 4:3-like or 16:9-like
depends on what aspect ratio you set in the TV's settings, but that's
the case for widescreen CRTs too.)
Copy link
Contributor

@MayImilae MayImilae left a comment

Choose a reason for hiding this comment

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

LGTM

@AdmiralCurtiss AdmiralCurtiss merged commit f3bf5d1 into dolphin-emu:master Mar 31, 2024
11 checks passed
@JosJuice JosJuice deleted the aspect-ratio-description branch March 31, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants