-
Notifications
You must be signed in to change notification settings - Fork 449
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
Change OK button on Advanced Preferences to Save button #601
Comments
Commented by Nicolas on 20 Feb 38310441 20:00 UTC Make sure translators also write the button label in a way that makes clear it's doing changes. |
Commented by Nicolas on 29 Oct 38310447 23:06 UTC |
Commented by Pepo on 11 May 38315316 09:46 UTC
|
Commented by Ageless on 17 Jan 39897348 17:46 UTC How difficult is it to use '''Save''' instead of '''OK'''? Or else, as interim solution add a good warning at the top. Something like "Clicking OK will save all preferences in this window to a local file, will override any of the same preferences set through the projects and BOINC will start using them immediately. If you do not want to use these local preferences, click Cancel." |
The Dialog still uses OK/Cancel buttons with a tooltip that says the changes will be saved. There is also awarning at the top of the page that says which preferences are used (local or web). I still find this not clear enough. Relabeling the button from OK to Save would be much clearer. |
Seeing how our developers are adding buttons to the manager's properties windows, can this issue be looked at now as well? |
Yes, I can do this
Best regards,
Vitalii Koshura
2017-08-15 0:15 GMT+03:00 Jord <notifications@github.com>:
… Seeing how our developers are adding buttons to the manager's properties
windows, can this issue be looked at now as well?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#601 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADFZoSFSQmV3KZPvh-_zcFVK-N4jcuf4ks5sYLjZgaJpZM4DbUjw>
.
|
Thanks, Vitalii. :-)
…-- Jord van der Elst.
On Mon, Aug 14, 2017 at 11:25 PM, Vitalii Koshura ***@***.***> wrote:
Yes, I can do this
Best regards,
Vitalii Koshura
2017-08-15 0:15 GMT+03:00 Jord ***@***.***>:
> Seeing how our developers are adding buttons to the manager's properties
> windows, can this issue be looked at now as well?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#601 (comment)>, or
mute
> the thread
> <https://github.com/notifications/unsubscribe-
auth/ADFZoSFSQmV3KZPvh-_zcFVK-N4jcuf4ks5sYLjZgaJpZM4DbUjw>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#601 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AKXbUzw3BCORpDOX1Byi3B4Yiz2aYkl3ks5sYLtdgaJpZM4DbUjw>
.
|
Originally requested in BOINC#601: Change OK button on Advanced Preferences to Save button. Button renamed in all dialogs with settings for consistency. Also now 'Cancel' button is default button. Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Fixed in #2043 |
Looks great - thanks. |
Note that if this is ported to the 7.8 branch, it will require another update to the localization template BOINC_Manager.pot. It is important that any changes to localizable strings be done long enough before public release to give the translators for the various languages sufficient time to update their translations.
|
The label of this button now disagrees with the instructions at the top of the dialog in clientgui/DlgAdvPreferencesBase.cpp and clientgui/sg_DlgPreferences.cpp, which still say to "click OK" when there is no global_prefs_override.xml file:
Using web-based preferences from <project URL>
Set values and click OK to use local preferences instead
|
@CharlieFenton, thank you. I'll fix this
Best regards,
Vitalii Koshura
2017-08-18 11:12 GMT+03:00 CharlieFenton <notifications@github.com>:
… The label of this button now disagrees with the instructions at the top of
the dialog in clientgui/DlgAdvPreferencesBase.cpp and
clientgui/sg_DlgPreferences.cpp, which still say to "click OK" when there
is no global_prefs_override.xml file:
Using web-based preferences from <project URL>
Set values and click OK to use local preferences instead
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#601 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADFZoV7czbdtrcTUVZhDhs97hC9DLwolks5sZUeAgaJpZM4DbUjw>
.
|
Change text labels to remove inconsistency betweeb them and button text as described in BOINC#601 Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
Fixed in #2050 |
So this is why when I upgraded from 7.8.3 to 7.9.3 and found that the |
Reported by Ageless on 17 Apr 38309470 22:13 UTC
When using the Simple View preferences, you go out by either clicking the Save or Cancel button. When using the Advanced View preferences, you go out by clicking either the OK or Cancel button.
People now just look into the Advanced Preferences and without knowing it make them work as they click OK to go out, then later they complain on forums that their preferences on the web site don't work when they change them and update BOINC. We increasingly find that these people are unknowingly using the override preferences, because they looked in once and went out by clicking OK.
So my idea is to standardize the buttons in Simple View and Advanced View, make them both Save and Cancel. Turning them around, so Cancel is first, would perhaps also make a difference. An extra pop-up window in which you have to confirm that you want to use the override preferences, may also do the trick.
Migrated-From: http://boinc.berkeley.edu/trac/ticket/623
The text was updated successfully, but these errors were encountered: