-
Notifications
You must be signed in to change notification settings - Fork 467
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
Force HTML highlighting for .vtl files #28084
Labels
Gardening Day
LTS: Excluded
Ticket that has been excluded from at least one LTS
Merged
OKR : User Experience
Owned by Jason
QA : Approved
QA : Passed Internal
Release : 24.04.24 v6
Included in LTS patch release 24.04.24 v6
Release : 24.07.19
Bug Fixing
Team : Lunik
Team : Scout
Triage
Type : New Functionality
Comments
wezell
added
Type : New Functionality
Triage
OKR : User Experience
Owned by Jason
Team : Lunik
labels
Mar 26, 2024
valentinogiardino
added a commit
that referenced
this issue
Jul 9, 2024
valentinogiardino
added a commit
that referenced
this issue
Jul 10, 2024
valentinogiardino
added a commit
that referenced
this issue
Jul 10, 2024
github-merge-queue bot
pushed a commit
that referenced
this issue
Jul 10, 2024
### Proposed Changes * Introduced logic to set the editor's language to HTML when editing VTL files: 1. Extracted file extension logic into extractFileExtension. 2. Created setVelocityLanguage for .vtl file handling. 3. Moved language update logic for other extensions to updateLanguageForFileExtension. 4. Extracted validateFileType logic to check file types in all cases. ### Screenshots ![html-highlighting-for-vtl-files](https://github.com/dotCMS/core/assets/77643678/04aa6902-487a-4536-8222-820edae4c453)
bryanboza
added
QA : Approved
Release : 24.07.12
Bug Fixing
and removed
QA : Approved
Release : 24.07.12
Bug Fixing
labels
Jul 11, 2024
IQA PassedDocker Image: ScreenshotScreen.Recording.2024-07-15.at.11.19.17.AM.mov |
erickgonzalez
added
Next LTS Release
Shortlisted of issues that will be included in the upcoming LTS
LTS: Excluded
Ticket that has been excluded from at least one LTS
labels
Jul 18, 2024
Only for 24.04 LTS |
gortiz-dotcms
added a commit
that referenced
this issue
Jul 25, 2024
erickgonzalez
added
Release : 24.04.24 v6
Included in LTS patch release 24.04.24 v6
and removed
Next LTS Release
Shortlisted of issues that will be included in the upcoming LTS
labels
Jul 31, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Gardening Day
LTS: Excluded
Ticket that has been excluded from at least one LTS
Merged
OKR : User Experience
Owned by Jason
QA : Approved
QA : Passed Internal
Release : 24.04.24 v6
Included in LTS patch release 24.04.24 v6
Release : 24.07.19
Bug Fixing
Team : Lunik
Team : Scout
Triage
Type : New Functionality
User Story
When editing a VTL using the new monaco editor in the system, it is important that I should get good syntax highlighting on my files. As there is no VTL setting for monaco, we should force the HTML syntax onto VTL files.
This is how it looks now:
Acceptance Criteria
I should see HTML color coded when looking at a VTL file
Proposed Objective
Technical User Experience
Proposed Priority
Priority 2 - Important
External Links... Slack Conversations, Support Tickets, Figma Designs, etc.
Assumptions & Initiation Needs
No response
Quality Assurance Notes & Workarounds
No response
Sub-Tasks & Estimates
No response
The text was updated successfully, but these errors were encountered: