Skip to content

Conversation

@juliasilge
Copy link
Collaborator

@juliasilge juliasilge commented Oct 8, 2025

Addresses #845

This PR undoes the change in 6b416d3 that accidentally set up onDidSaveTextDocument that applies to all documents. This bug was in a code path that only applied to VS Code.

@juliasilge juliasilge changed the title Undo onDidSaveTextDocument that causes problems for _all_ files Undo onDidSaveTextDocument that causes problems for _all_ files in VS Code Oct 8, 2025
@juliasilge juliasilge changed the title Undo onDidSaveTextDocument that causes problems for _all_ files in VS Code Undo onDidSaveTextDocument that causes problems for ALL files in VS Code Oct 8, 2025
@juliasilge
Copy link
Collaborator Author

We can confirm the fix by getting the artifact from here: https://github.com/quarto-dev/quarto/actions/runs/18350404038/job/52268505072?pr=848

Download, unzip, then Extensions: Install from VSIX...

@juliasilge juliasilge requested review from cderv and cscheid October 8, 2025 15:56
@juliasilge juliasilge marked this pull request as ready for review October 8, 2025 15:56
@juliasilge
Copy link
Collaborator Author

You should still be able to switch between visual and source mode in VS Code, but you should not see any attempts to open in visual mode in non-Quarto files on a save.

Copy link
Contributor

@cderv cderv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tested the build from CI. It is working now, I can't reproduce the issue anymore

@juliasilge juliasilge merged commit 16ead3c into main Oct 8, 2025
2 checks passed
@juliasilge juliasilge deleted the fix-vscode-on-did-save branch October 8, 2025 16:12
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

Successfully merging this pull request may close these issues.

2 participants