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

Improve wording in release notes & in the toggle to make CrossOver not auto update #189

Open
rxhfcy opened this issue Apr 11, 2024 · 10 comments

Comments

@rxhfcy
Copy link

rxhfcy commented Apr 11, 2024

This page: https://github.com/italomandara/CXPatcher/releases/tag/v0.5 has the following (see screen shot)

What does this mean

Question:

What does "OR DISABLE THIS" mean in this context?

Disable the "Disable" toggle?
or
Enable the "Disable" toggle?

Suggestion:

Could the toggle text itself + the help text in the "release notes" be made more explicit/clear please?

For the toggle: Something like "Make CrossOver not auto update", or whatever the toggle does/means?

For the text: "Change this toggle to "enabled" (or "disabled"), to make it look like this:" or something

@rxhfcy
Copy link
Author

rxhfcy commented Apr 16, 2024

v0.5.1 is much better, but to nitpick:

  • In the pref itself, it should say "CrossOver" with a capitalized "O", not "Crossover"
  • In the pref itself, it's still weird that you "enable" a switch to "disable" and vice versa...
Better

@rxhfcy rxhfcy changed the title Unambiguous wording: What does "OR DISABLE THIS:" mean in v0.5 "release notes"? Improve wording in release notes & in the toggle to make CrossOver not auto update Apr 17, 2024
@italomandara
Copy link
Owner

Any suggestions? "Prevent CrossOver auto update" maybe?

@rxhfcy
Copy link
Author

rxhfcy commented Apr 21, 2024

Any suggestions? "Prevent CrossOver auto update" maybe?

Sounds fine to me (pretty minimal change but would be much better I think).

@italomandara
Copy link
Owner

ok done

@italomandara
Copy link
Owner

bc4685c

@rxhfcy
Copy link
Author

rxhfcy commented Apr 30, 2024

BTW doesn't this have a similar problem? The user has to "enable" this in order to "disable" something?

"autoUpdateToggleHelp" = "Disable CrossOver's update check prompt";

(I don't actually understand what the string means in context, but)
-> "Prevent CrossOver something something"?

@italomandara
Copy link
Owner

italomandara commented May 1, 2024

Crossover has an auto update feature and it's part of the app, you can't disable it normally, the toggle enables a hack that removes the ability to auto update.

@rxhfcy
Copy link
Author

rxhfcy commented May 1, 2024

"Enable hack to remove CrossOver auto update feature"? :)

My point is that it can feel somewhat silly to enable to disable

@Stromweld
Copy link

It's a switch so can think of it as on/off or true/false instead of enable/disable. switch on for disable auto update = true.

@rxhfcy
Copy link
Author

rxhfcy commented May 1, 2024

Sure, just a random comment

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

No branches or pull requests

3 participants