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.

Sign up for GitHub

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

Jump to bottom

[UD] Migrating from Monaco Editor to CodeMirror #22481

Closed
Tracked by #22243
olexii4 opened this issue Sep 6, 2023 · 0 comments · Fixed by eclipse-che/che-dashboard#945
Closed
Tracked by #22243

[UD] Migrating from Monaco Editor to CodeMirror #22481

olexii4 opened this issue Sep 6, 2023 · 0 comments · Fixed by eclipse-che/che-dashboard#945
Assignees
@olexii4
Labels
area/dashboard kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che

Comments

@olexii4
Copy link
Contributor

olexii4 commented Sep 6, 2023

Is your enhancement related to a problem? Please describe

After resolving the issue Make "Devfile" on User Dashboard uneditable, Monaco Editor dependency looks pretty extensive as for read-only tool.

We should migrate from Monaco Editor to CodeMirror to decrease bundle size.

Describe the solution you'd like

We should remove the next libraries: "@eclipse-che/workspace-client", "yaml-language-server" and "monaco-languageclient". Then migrate from Monaco Editor to CodeMirror.

Describe alternatives you've considered

No response

Additional context

No response
@olexii4 olexii4 added kind/enhancement A feature request - must adhere to the feature request template. area/dashboard labels Sep 6, 2023
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Sep 6, 2023
@ibuziuk ibuziuk added the severity/P2 Has a minor but important impact to the usage or development of the system. label Sep 6, 2023
@ibuziuk ibuziuk moved this to 📅 Planned in Eclipse Che Team A Backlog Sep 6, 2023
@dmytro-ndp dmytro-ndp added team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Sep 6, 2023
@ibuziuk ibuziuk mentioned this issue Sep 7, 2023
Closed
45 tasks
@ibuziuk ibuziuk moved this from 📅 Planned to 📋 Backlog in Eclipse Che Team A Backlog Sep 27, 2023
@olexii4 olexii4 mentioned this issue Oct 9, 2023
Merged
@svor svor moved this from 📋 Backlog to ✅ Done in Eclipse Che Team A Backlog Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olexii4 olexii4

Labels
area/dashboard kind/enhancement A feature request - must adhere to the feature request template. severity/P2 Has a minor but important impact to the usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: migrate from Monaco to CodeMirror eclipse-che/che-dashboard
4 participants
@dmytro-ndp @ibuziuk @olexii4 @che-bot