- Notifications
You must be signed in to change notification settings - Fork 2
Extend HTML#8
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
Extend HTML #8
Uh oh!
There was an error while loading. Please reload this page.
Conversation
deathaxe commented Feb 6, 2025 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
deathaxe commented Feb 14, 2025
Any objections about moving the package to SublimeText org? |
nwoltman commented Feb 15, 2025
This is great. Thanks @deathaxe!
It's possible to enter embedded JS with the
Not at all! If we're going to do that, we may want to coordinate with the maintainer of the https://packagecontrol.io/packages/EJS package so we can use the "EJS" package name (instead of "EJS 2"). I originally made this package because that one had gotten outdated and it seemed like it was unmaintained. We did talk about transferring the "EJS" name to my package but that never ended up happening. |
deathaxe commented Feb 15, 2025
Already put a little more work on it at https://github.com/deathaxe/sublime-ejs/tree/st4152 (lazily squshing all changes) to follow the strategy we already use for PHP, which (as EJS) is a template engine which doesn't care about underlying (HTML) syntaxes. Mixing two context intensive syntaxes (e.g. JS and CSS) is hard, so we have to make compromizes, but it should work for most real world use cases. For now, I kept only the most common tag names
Note, that I happily add you as a maintainer so you still will have full access to your repo. Github forwards any request from your current repo to any new location, so moving and updating index doesn't need to take place a the same point of time. |
This commit ... 1. replaces `push...with_prototype` by extending HTML, CSS and JavaScript. 2. adds support for $TEMPLATE_START and $TEMPLATE_END snippet variables, to avoid duplicated snippet variables or key bindings. 3. adds support to toggle `<%# %>` comments 4. removes `<$ $>` and `<@ @>` variants, keeping only most commonly used ones. Actually EJS supports anything (even `[% %]` or `{% %}` all of which is impossible to support at the same time via static syntax definitions. It would make more sense to dynamically create them via templates. 5. drops Dracula Color scheme as color schemes should not be bundled with syntax definitions and syntax only uses scopes according to ST's scope naming guide. It should be easily possible for original Dracula color scheme to support them.nwoltman commented Feb 17, 2025
@deathaxe I looked at your branch. I'm happy to move this package to the Sublime Text org. Please let me know if there's anything I need to do to help facilitate this. |
deathaxe commented Feb 17, 2025
You'd have to initiate the transfer at the bottom of repository settings, via "Transfer Ownership" button. It should either directly move the repo or send a confirmation email to SublimeText admins to accept the move. |
nwoltman commented Feb 25, 2025
I tried transferring ownership to the
Not sure if you got any notification for this. Some permissions might need to be changed on your end to allow me to transfer to the repo to your org. |
deathaxe commented Feb 25, 2025
Community packages are organized under https://github.com/SublimeText. This is where EJS should find its new home. The "error" message you see when initiating ownership transfer, just indicates an admin of the target organization to need confirm the transfer. As soon as it is confirmed, transfer is started. |
nwoltman commented Feb 26, 2025
Ah thank you. I've now initiated a transfer to the SublimeText org. |
deathaxe commented Feb 26, 2025
Sent you an invitation to SublimeText org to enable transfer and later on being able to assign you moderator rights to your repo again. Seems I was on wrong track with just approving being enough. |
nwoltman commented Feb 27, 2025
OK everything seems to be good now. It's been moved. |
d58e79e into SublimeText:masterUh oh!
There was an error while loading. Please reload this page.
Fixes#5
Fixes#6
Fixes#7
This PR replaces
push...with_prototypeby extending ST's HTML syntaxMain scope is changed from
text.ejstotext.html.ejsin order to also support all HTML related completions, snippets and behavior within EJS templates.Note
It supports
<%...%>within HTML, but not in<script>or<style>tags. Not familiar with EJS to judge whether that's needed/supported.