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

Formatter should show notification modal instead of blocking dialog if there are no formatters installed #132032

Closed
yunusefendi52 opened this issue Sep 1, 2021 · 4 comments
Assignees
Labels
formatting Source formatter issues *out-of-scope Posted issue is not in scope of VS Code under-discussion Issue is under discussion for relevance, priority, approach
Milestone

Comments

@yunusefendi52
Copy link

In previous version (1.55.2) if there are no formatters installed it shows notification modal instead of blocking dialog, but the latest version (1.60.0-insider) it shows dialog. The reason I am not a fan of dialog in this case is because it prevents from typing and I have to close the dialog before continue typing, here is examples from v1.55.2 and the latest version (1.60.0-insider)

vscode v1.55.2 shows notification modal
upload1

vscode v1.60.0-insider shows dialog, which prevents from typing
Screen Recording 2021-09-01 at 1 34 14 PM

context: #119463 (comment) and related issue #119463

cc @isidorn

@isidorn
Copy link
Contributor

isidorn commented Sep 3, 2021

@yunusefendi52 this is fair feedback and thanks for the nice gifs.
However the previous experience was not nice either: the notification was always popping out.
I hope that in the new experience you press cancel once and the dialog no longer appears ever?

I would suggest to keep this open for now and if we get more feedback before reacting.

fyi @jrieken

@isidorn isidorn added formatting Source formatter issues under-discussion Issue is under discussion for relevance, priority, approach labels Sep 3, 2021
@isidorn isidorn added this to the Backlog milestone Sep 3, 2021
@yunusefendi52
Copy link
Author

yunusefendi52 commented Sep 4, 2021

I hope that in the new experience you press cancel once and the dialog no longer appears ever?

I think cancel should be meant only for cancelling the dialog and not having other side effects
but if I can pick both, I would have pick both but with some sort of settings to decide which one should be showed (notification or dialog), I am not sure if that's going to clutter the settings page though

@isidorn
Copy link
Contributor

isidorn commented Sep 6, 2021

We will definitely not add a setting to control if there is a modal dialog or a notification. We should figure out what is the better experience and stick with it. Currently we are leaning towards modal dialogs.

@isidorn isidorn added the *out-of-scope Posted issue is not in scope of VS Code label Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
formatting Source formatter issues *out-of-scope Posted issue is not in scope of VS Code under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

3 participants
@isidorn @yunusefendi52 and others