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

Implemented in monaco-editor #8

Open
cjboy76 opened this issue Oct 9, 2022 · 3 comments · May be fixed by #16
Open

Implemented in monaco-editor #8

cjboy76 opened this issue Oct 9, 2022 · 3 comments · May be fixed by #16

Comments

@cjboy76
Copy link

cjboy76 commented Oct 9, 2022

It is working great so far,
Is it possible to work with monaco-editor but not monaco-editor-core ?
Or any way that I could register "vue" language in monaco-editor ?

@Kingwl
Copy link
Owner

Kingwl commented Oct 14, 2022

Well. I'm not pretty sure the correct way.

Friendly ping @orta. Would you mind give some suggestions?

@orta
Copy link

orta commented Oct 14, 2022

Never needed to use monaco-editor-core, from a glance it's the underlaying data models for monaco-editor - I think if you're working with those levels of primitives it's quite reasonable that you should be able to give a PR here to get this repo working with it

Ah they want to know the high level monaco-editor APIs for this sort of stuff, yeah, it'd need to register using monaco.languages.register({ id: "vue", }) and then set up the LSP and workers from that, you can refer to https://github.com/microsoft/monaco-editor/tree/22698f2c1b65020c7fd9375739d3072c704dd5a4/src/language/json as a simple example

@Deuscx Deuscx linked a pull request Feb 4, 2023 that will close this issue
@denys119
Copy link

denys119 commented Mar 6, 2023

I am also interested if it is possible to work with monaco-editor instead of monaco-editor-core. Maybe this PR #16 will work for this issue.

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 a pull request may close this issue.

4 participants