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

Prefer usage of dialogs without ? for confirmation dialogs in PDE #149

Closed
vogella opened this issue Jun 9, 2022 · 14 comments
Closed

Prefer usage of dialogs without ? for confirmation dialogs in PDE #149

vogella opened this issue Jun 9, 2022 · 14 comments

Comments

@vogella
Copy link
Contributor

vogella commented Jun 9, 2022

Using a ? icon for confirmation dialogs is not recommended in the lateset UI guidelines for Mac and Linux and I do not see that used in Windows.

image

References:
https://developer.apple.com/design/human-interface-guidelines/components/presentation/alerts/
https://developer.gnome.org/hig/patterns/feedback/dialogs.html

I suggest we do not use ? icons anymore in our PDE dialogs whenever possible.

@merks
Copy link
Contributor

merks commented Jun 9, 2022

I'm not sure I follow what these links are suggesting nor what changes you are actually suggesting.

I do wonder how those recommendations correspond to these guidelines

https://wiki.eclipse.org/User_Interface_Guidelines

Will this approach/change make PDE different from say JDT or the Platform and all the other projects?

@vogella
Copy link
Contributor Author

vogella commented Jun 9, 2022

The way we currently work is that we start somewhere and continue improving.

@merks
Copy link
Contributor

merks commented Jun 9, 2022

That doesn't answer the question though. Could you explain the improvement, perhaps with a specific example of what you plan to change so that someone other than you can understand what you plan to do and what you consider an improvement. After all, if it's an improvement, everyone should do it and then everyone is likely to have an opinion about it...

@vogella
Copy link
Contributor Author

vogella commented Jun 9, 2022

@akurtakov I think you have screenshots from the native behavior? Can you share them here?

@akurtakov
Copy link
Member

Screenshot from 2022-06-09 10-55-22
Screenshot from 2022-06-09 10-54-40
This is the pure SWT MessageBox, which matches exactly what one sees in native GTK apps.

@merks
Copy link
Contributor

merks commented Jun 9, 2022

This is pure native Windows 10:

image

This uses a question mark. So if this were PDE, it would be changed how? One doesn't normally say yes/no in response to a statement but rather in response to a question, which is generally terminated by a question mark. So is just the question mark removed or is a yes/no dialog deprecated?

@vogella
Copy link
Contributor Author

vogella commented Jun 9, 2022

The issue is only to remove the ? mark from the dialog which is not included in the WIndows UI guidelines (I finally found it): https://docs.microsoft.com/en-us/windows/apps/design/controls/dialogs-and-flyouts/dialogs

Yes / No dialogs are still fine even though I think action orientated buttons are preferrable.

@akurtakov
Copy link
Member

akurtakov commented Jun 9, 2022

The goal should be to use SWT's MessageBox directly as it already follows the underlying OS guidelines and thus Eclipse shows dialog that doesn't differ from what rest of the apps on the system show.
Regarding the "One doesn't normally say yes/no in response to a statement but rather in response to a question," this is native message box with artificial text for displaying purposes, the text itself doesn't make sense either.

@vogella
Copy link
Contributor Author

vogella commented Jun 9, 2022

Screenshot from the UI guidelines of Microsoft.

image

@merks
Copy link
Contributor

merks commented Jun 9, 2022

That dialog has three question marks (oops I mean two). Is that bad?

@akurtakov
Copy link
Member

Our goal should be to defer to the OS as SWT was initially designed - "The primary design goals are high performance, native look and feel, and deep platform integration. "

@vogella
Copy link
Contributor Author

vogella commented Jun 9, 2022

Ah, sorry @merks I mean the question mark icon on the left side of the dialog. Sorry for not being clear, it was obvious to my mind as I head multiple times about this in the last years. I added a screensho to the intial comment to make it clear to others.

@merks
Copy link
Contributor

merks commented Jun 9, 2022

I see now where the wires got crossed! Thanks for clarifying.

vogella added a commit that referenced this issue Jun 17, 2022
Using a help icon for confirmation dialogs is not recommended in the
latest UI guidelines for Mac, Linux and Windows.

References:
https://developer.apple.com/design/human-interface-guidelines/components/presentation/alerts/
https://developer.gnome.org/hig/patterns/feedback/dialogs.html
https://docs.microsoft.com/en-us/windows/apps/design/controls/dialogs-and-flyouts/dialogs

Currently the text is way to much on the left with this patch, therefore
this change depends on
eclipse-platform/eclipse.platform.ui#116

For #149
@vogella
Copy link
Contributor Author

vogella commented Jun 22, 2022

Done for all dialogs I regulary see, will open new PR if I see others which can be updated.

@vogella vogella closed this as completed Jun 22, 2022
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