-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
editor.maxTokenizationLineLength is not used leading to bogus coloring #7617
Comments
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
Do I need to change something in the oml-vscode extension or in the opencaesar/vocabularies repo to get this fix? |
@melaasar Hi 👋 We limit tokenisation to max 400 characters by default since it can hang the browser for big files. But a user can change this default with |
Thanks. I went ahead and added a workspace setting file where I added a value of 800 to editor.maxTokenizationLineLength: However, when I start a new workspace, and browse to https://github.com/opencaesar/vocabularies/blob/master/src/imce.jpl.nasa.gov/foundation/base.oml |
@melaasar It was not deployed to Gitpod yet. It was only merged for Theia. Check out it tomorrow or on Friday maybe it will be there. |
Reproducible with https://github.com/opencaesar/vocabularies/blob/9b8c1ff9d07803420cebe03f44687a4da253373b/src/imce.jpl.nasa.gov/foundation/base.oml and https://github.com/opencaesar/oml-vscode
The text was updated successfully, but these errors were encountered: