Skip to content

Various typos needed fixing #3554

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

Closed
waydabber opened this issue Oct 28, 2024 · 2 comments
Closed

Various typos needed fixing #3554

waydabber opened this issue Oct 28, 2024 · 2 comments
Assignees
Labels
bug Something isn't working done All tasks are completed released Released
Milestone

Comments

@waydabber
Copy link
Owner

waydabber commented Oct 28, 2024

Hi waydabber, it's @niklasbogensperger from the German translation thread 👋

Like I mentioned in the comment thread on GitHub, here is my list of minor corrections in the original strings:

If source and target control ranges differ, synchronized values are remaped (rebased) accordingly.
-> "remapped"

MacOS can mute digital streams natively [...]
-> "macOS" (lowercase even at start of sentence)

Newly connected displays are normally treated as possible affected displays only. [...]
-> "possibly" (?)

Phyiscal dimensions are used by the app for UI scale matching [...]
-> "Physical"

[...] This lets change GUI size (display scaling) freely via the resolution slider.
-> "This lets you"

[...] Helps if a display is strandled disconnected in case of an app crash of forced quit.
-> "stranded" / "stranded as"
-> "crash or forced"

Select one or more filters to specify which displays are allowed be sync sources for this configuration!
-> "allowed to be"

Show currently off-line displays in Settings
[...] Allows some configuration changes even when a display is off-line.
-> "offline" (consistency)

The app menu will show this name instead of the system name in the app menu for the display. [...]
-> app menu twice?

The app was launched using the Rosetta. [...]
-> "using Rosetta"

[...] Physically reconnect  display to revert back to the original in case of serious trouble.
-> double space after "reconnect"

[...] Alternatively you can disable auto-connect (it can be enable any time under association settings).
-> "be enabled"
There are no at least two display group members present.
There is no at least one display group member present.
-> "There is not" / "There are not"

These three identifiers (serial number, vendor and model ID) serves as the basis [...]
-> "serve"

Let the application control ... [...] (x2)
-> "app" (consistency)

A pop-up notification will appear when the app detects [...]
-> "popup" (consistency)

[...] Use only if you want to finetune the dimming curve below the configured XDR brightness scale minimum
-> "fine-tune" (consistency and recommended spelling)

%@%%
%@Hz
%.3fHz
%@mm
%1$@mm x %2$@mm
%1$lld x %2$lld %3$@ @ %4$lldHz
-> missing spaces? (SI) units should always be preceded by a space (except for º (degree), ' (arcmin), " (arcsec))

Screen Rotation%@
-> missing space?

Apple Silicon Macs tend freeze the display instead of letting [...]
-> "tend to"

By default the keys controls change one OSD chiclet worth of value [...]
-> "the key controls" / "the keys change" / "the controls change" ? (sentence structure is confusing)
@waydabber waydabber added bug Something isn't working done All tasks are completed internal pre-release Enable internal pre-releases under Settings/Applications/Updates labels Oct 28, 2024
@waydabber waydabber added this to the TBD milestone Oct 28, 2024
@waydabber waydabber self-assigned this Oct 28, 2024
@niklasbogensperger
Copy link

[...] Helps if a display is strandled disconnected in case of an app crash of forced quit.

In the newest version of the strings the extra "l" in stranded is still there, otherwise it's all fixed (I'm assuming the unit spacing is/has to be handled differently?)

@waydabber
Copy link
Owner Author

Oh sorry, will fix that, thank you.

The `Screen Rotation%@' is good as it is as the space is added by the app before the degree which must always be at the end of the string.

The units are ambiguous. I could make a change that but would raise various compatibility issues as some of the values are actually persisted as a string for various reasons and if I make a change it might mess things up. I looked up on the net how prevalent '60Hz' is versus '60 Hz' and to me it seems like in the context of computer monitors omitting the space is actually quite common and accepted (at least it does not hurt my eyes). I asked ChatGPT about it - it says it is better to use "60Hz" as this is how people are used to it, but if I write a scientific paper or in a formal setting, it might indeed be better to add a space to align with SI standards.

@waydabber waydabber changed the title Various typos needs fixing Various typos needed fixing Nov 6, 2024
@waydabber waydabber added pre-release available Released as beta version and removed internal pre-release Enable internal pre-releases under Settings/Applications/Updates labels Nov 22, 2024
@waydabber waydabber added released Released and removed pre-release available Released as beta version labels Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working done All tasks are completed released Released
Projects
None yet
Development

No branches or pull requests

2 participants