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

Replace all Yes/No/OK/Cancel dialogs #149

Open
oscargus opened this issue Aug 5, 2016 · 12 comments
Open

Replace all Yes/No/OK/Cancel dialogs #149

oscargus opened this issue Aug 5, 2016 · 12 comments

Comments

@oscargus
Copy link

oscargus commented Aug 5, 2016

As brought up by @Siedlerchr in JabRef#1683 http://ux.stackexchange.com/questions/756/what-are-some-alternatives-to-the-phrase-are-you-sure-you-want-to-xyz-in-confi

We currently have 50 instances of JOptionPane.showConfirmDialog where for sure this is not followed.

@Siedlerchr
Copy link
Member

I already documented this in our wiki, too ;)

@oscargus
Copy link
Author

oscargus commented Aug 5, 2016

Yes, but good to have issues if we should change the code, not just write good new code. ;-)

It is, however, not enough to only search for showConfirmDialog:

                int answer = JOptionPane.showOptionDialog(KeyBindingsDialog.this,
                        Localization.lang("All key bindings will be reset to their defaults.") + " " +
                        Localization.lang("Continue?"),
                        Localization.lang("Resetting all key bindings"),
                        JOptionPane.YES_NO_OPTION,
                        JOptionPane.QUESTION_MESSAGE, null,
                        new String[] {Localization.lang("OK"),
                                Localization.lang("Cancel")},
                        Localization.lang("OK"));

@Siedlerchr
Copy link
Member

@oscargus Maybe you could add some sample code for a correct dialog in the wiki section?

@Siedlerchr
Copy link
Member

Windows Design Guidelines also have a section about this: https://msdn.microsoft.com/de-de/library/windows/desktop/dn742470.aspx

@koppor
Copy link
Member

koppor commented Aug 29, 2021

@koppor
Copy link
Member

koppor commented Oct 25, 2021

Example dialog

grafik

@koppor
Copy link
Member

koppor commented Oct 18, 2023

For reference, this is how KeePass does it; which we DO NOT want to do:

grafik

@colinldbd
Copy link

Hi, I'm a new contributor and for a university assignment I would like to learn how to contribute to an open-source project. I'll read through the contributor's guide carefully and give this issue a go.

Can you please assign this issue to me? I also commented on another first good issue, maybe I'll give them both a try if possible.

@SUPWEZZ
Copy link

SUPWEZZ commented Oct 20, 2023

Hi @koppor, I would like the work on the issue, please assign it to me if possible!

@koppor
Copy link
Member

koppor commented Oct 20, 2023

@SUPWEZZ @colinldbd was quicker than you, so I assign @colinldbd

@koppor koppor moved this from Free to take to Reserved in Good First Issues Oct 20, 2023
@colinldbd
Copy link

@Siedlerchr @oscargus Hi, I see that you might have added some sample code for a correct dialog in the wiki section, can I get a link to where that is?
Many thank!

@koppor
Copy link
Member

koppor commented Oct 22, 2023

We only have some hints at https://devdocs.jabref.org/code-howtos/ui-recommendations.html. There, we link to https://ux.stackexchange.com/questions/756/what-are-some-alternatives-to-the-phrase-are-you-sure-you-want-to-xyz-in-confi/768#768 as an example to replace "OK/Cancel" by "Post/Cancel"

Note, the hard task is to find the dialogs, understand their context and to rename the buttons. The easy task is then to implement the renamings. -- This issue covers the hard part, too. In case this is too hard, be invited to check other issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants