-
Notifications
You must be signed in to change notification settings - Fork 24
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
Comments
v0.5.1 is much better, but to nitpick:
![]() |
Any suggestions? "Prevent CrossOver auto update" maybe? |
Sounds fine to me (pretty minimal change but would be much better I think). |
ok done |
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) |
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. |
"Enable hack to remove CrossOver auto update feature"? :) My point is that it can feel somewhat silly to enable to disable |
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. |
Sure, just a random comment |
This page: https://github.com/italomandara/CXPatcher/releases/tag/v0.5 has the following (see screen shot)
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
The text was updated successfully, but these errors were encountered: