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

Format on save no longer working if more than 1 formatter is available #70965

Closed
eldair opened this issue Mar 22, 2019 · 7 comments
Closed

Format on save no longer working if more than 1 formatter is available #70965

eldair opened this issue Mar 22, 2019 · 7 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@eldair
Copy link

eldair commented Mar 22, 2019

Issue Type: Bug

Format on save no longer working if more than 1 formatter is available.
Running format document command gives a list of formatters and by selecting the one I want it works. Is there some new option for defining default formatter?

VS Code version: Code - Insiders 1.33.0-insider (9474154, 2019-03-22T06:18:30.063Z)
OS version: Windows_NT x64 10.0.18362

System Info
Item Value
CPUs AMD Ryzen 7 1700X Eight-Core Processor (16 x 3394)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: enabled
surface_synchronization: enabled_on
video_decode: enabled
webgl: enabled
webgl2: enabled
Memory (System) 15.93GB (3.02GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (34)
Extension Author (truncated) Version
better-comments aar 2.0.3
Bookmarks ale 10.3.0
color-info bie 0.5.0
vscode-intelephense-client bme 1.0.8
npm-intellisense chr 1.3.0
path-intellisense chr 1.4.2
laravel-blade cjh 1.1.2
bracket-pair-colorizer-2 Coe 0.0.28
vscode-svgviewer css 2.0.0
vscode-eslint dba 1.8.2
vscode-babel-coloring dza 0.0.4
gitlens eam 9.5.1
vscode-phpmd eco 1.1.0
EditorConfig Edi 0.13.0
json-tools eri 1.0.2
prettier-vscode esb 1.8.1
auto-close-tag for 0.5.6
phpcs ika 1.0.5
autofilename Jer 0.1.3
vscode-phpfmt kok 1.0.30
rainbow-csv mec 1.0.0
dotenv mik 1.0.1
vscode-scss mrm 0.6.2
atom-keybindings ms- 3.0.6
debugger-for-chrome msj 4.11.3
vetur oct 0.17.0
material-icon-theme PKi 3.6.3
polacode pnp 0.2.2
sass-indented rob 1.5.1
code-settings-sync Sha 3.2.7
vscode-fileutils sle 2.13.5
vscode-todo-highlight way 1.0.4
change-case wma 1.0.0
JavaScriptSnippets xab 1.7.2

(7 theme extensions excluded)

@vscodebot
Copy link

vscodebot bot commented Mar 22, 2019

@vscodebot vscodebot bot added the terminal Integrated terminal issues label Mar 22, 2019
@iansan5653
Copy link
Contributor

Related discussion here: #41882 (comment)

@Tyriar Tyriar assigned jrieken and unassigned Tyriar Mar 22, 2019
@Tyriar Tyriar removed the terminal Integrated terminal issues label Mar 22, 2019
@jrieken
Copy link
Member

jrieken commented Mar 25, 2019

/duplicate of #41882 (comment)

@vscodebot vscodebot bot added the *duplicate Issue identified as a duplicate of another issue(s) label Mar 25, 2019
@vscodebot
Copy link

vscodebot bot commented Mar 25, 2019

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Mar 25, 2019
@jrieken
Copy link
Member

jrieken commented Mar 25, 2019

@eldair Because format on save is without UI we cannot show a picker. We can continue to pick some formatter but IMO it would be highly inconsistent. We are open for suggestions but the goal is that users configure formatters so that only one is active per file

@jrieken
Copy link
Member

jrieken commented Mar 25, 2019

fyi - I have pushed a change that picks the first formatter for format on save. when this happens a status bar message is being printed, telling you which formatter has been picked

@eldair
Copy link
Author

eldair commented Mar 25, 2019

Hi,

Thank for the info and the update. I understand what the issue is, I thought there would be an option to select a formatter per language vs disable on extension which offer it as side effect (if a sole purpose of an extension is formatting than I agree it should be disabled if not in use).
Just an idea: dropdowns for all languages with a list of available formatters for selecting a desired one.

@vscodebot vscodebot bot locked and limited conversation to collaborators May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

4 participants