formatonsave moves cursor at end of file and unfolds every section

[t-fritsch]

Environment

• VSCode Version: 1.24.1
• OS Version: Win 10

Actual behavior

When formatonsave is activated, saving the file places the cursor at the very end of the file, forcing the user to manually move the cursor back to its initial position.
The folded sections also gets unfolded, which makes it even more difficult to go back to initial position when working on large css files

Expected behavior

when formatonsave is activated, saving the file should not change the cursor position, it should remain on the line that it was on.

Steps to reproduce

1. configure vscode csscomb to formatonsave
2. create a css file and type css code into it
3. save file
   

Config

"csscomb.formatOnSave": true,

Note:
this issue is there because a previous issue was closed by the reporter but never worked on : #110

Edit : this issue doesn't occur when formatOnSave is false and you run command "CSSComb: Format styles" from command palette (CTRL+SHIFT+p).

[muuvmuuv]

@mrmlnc really need a fix for this... it is annoying. Do you have any clue what is doing this? If you have no time and know where to start, I will try my best to make a PR to fix this.

[muuvmuuv]

This might help: microsoft/vscode#111

[nealoke]

@tfritsch-km any way I can help solve this?

[t-fritsch]

Sorry @nealoke, I really don't know.
Maybe ask @mrmlnc where to start (I haven't digged into the code of this plugin yet)

[nealoke]

Aah oops, I thought you were the author, @mrmlnc how can I help with this plugin? :)

[t-fritsch]

no problem 😄

[t-fritsch]

I tried to look at it, and noticed that the code that formats css is slightly different when saving file from the one that runs when formatting with the "CSSComb: Format styles" command :

on "CSSComb: Format styles" command :

vscode-csscomb/src/extension.ts

Line 69 in 65c71c4

builder.replace(block.range, block.content);

on save :

vscode-csscomb/src/extension.ts

Line 118 in 65c71c4

return vscode.TextEdit.replace(block.range, block.content);

I don't know the difference between the two, anyone that knows vscode api sees something interesting ?

[mrmlnc]

Hey,

Unfortunately, now I don't have free time to support this plugin. When I developed this plugin it was not possible to save the cursor position when we use event.waitUntil(actions). If you want to correct the current behavior, please send a pull request.

[t-fritsch]

Thank you @mrmlnc
I just did a PR, could you review it ? I haven't heavily tested it but it seems to work. Any advice on it ?

[t-fritsch]

Hey,
I've used the patch for about a week now, and it works fine so far, I didn't notice any regression. Would love to get some feedback.

[nealoke]

@mrmlnc could you please merge this?

[markconroy]

Hi,

I have tested the patch kumquats@8b16d53 and csscomb format on save is now working as expected.

It would be great to get this merged to the repo.

[nealoke]

🙏 merge please 😄