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

Respect language-specific cursorStyle setting #27

Merged
merged 1 commit into from Feb 21, 2022

Conversation

MrSimbax
Copy link

@MrSimbax MrSimbax commented Jan 8, 2022

Load configuration with language scope for the language
set in the active editor.

For example, if the following is in .vscode/settings.json

"editor.cursorStyle": "line",
"[lua]": {
    "editor.cursorStyle": "block",
}

then the cursor would be "line" instead of "block" in Lua files
when this extension is enabled. After this commit, it has "block" style.

The language-specific scope API was added in some more recent
version of VS Code. Tried to bump up the vscode package version
but turns out it's deprecated so replace the package with
@types/vscode. Remove the no longer needed (and working)
postinstall script. Update lock files which changed after
npm install.

Load configuration with language scope for the language
set in the active editor.

For example, if the following is in `.vscode/settings.json`

```json
"editor.cursorStyle": "line",
"[lua]": {
    "editor.cursorStyle": "block",
}
```

then the cursor would be "line" instead of "block" in Lua files
when this extension enabled. After this commit, it has "block" style.

The language-specific scope API was added in some more recent
version of VS Code. Tried to bump up the `vscode` package version
but turns out it's deprecated so replace the package with
`@types/vscode`. Remove the no longer needed (and working)
`postinstall` script. Update lock files which changed after
`npm install`.
@DrMerfy
Copy link
Owner

DrMerfy commented Jan 21, 2022

Hello, thanks for the request! This seems a cool feature, I'll test a bit locally and if everything seems good it will be passed into a release.

@DrMerfy
Copy link
Owner

DrMerfy commented Feb 21, 2022

@MrSimbax Sorry for the late reply, I have been real busy. I checked the commit and it works good.

@DrMerfy DrMerfy merged commit 2573866 into DrMerfy:master Feb 21, 2022
@DrMerfy
Copy link
Owner

DrMerfy commented Feb 21, 2022

@all-contributors please add @MrSimbax for code

@allcontributors
Copy link

@DrMerfy

I've put up a pull request to add @MrSimbax! 🎉

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.

None yet

2 participants