Skip to content

Conversation

@Bliztle
Copy link

@Bliztle Bliztle commented Apr 29, 2025

Added emmet lsp for html shorthand completions such as div>ul>li*3. This only adds a single, simple language module, contained to a single file.

I briefly considered if this should have been added as an extension to vim.languages.html, but decided on making it its own module since it also integrates with jsx and other html-like frontend frameworks. I have no strong feelings about his however.

If there is anything I've missed or need to change please do tell, and I will happily see to it!

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

github-actions bot pushed a commit that referenced this pull request Apr 29, 2025
@github-actions
Copy link

github-actions bot commented Apr 29, 2025

🚀 Live preview deployed from 4b904de

View it here:

Debug Information

Triggered by: Bliztle

HEAD at: emmet-language-server

Reruns: 1532

@Bliztle
Copy link
Author

Bliztle commented Apr 29, 2025

Should I add emmet to languages.md? I noticed quite a few languages was missing so I wasn't sure whether to update it or leave it be

@NotAShelf
Copy link
Owner

The languages.md is in a shaky position atm, do feel free to add Emmet as I'll probably update it with th upcoming documentation refactor.

@horriblename
Copy link
Collaborator

eslint is used by ts, astro, etc. and we didn't add a dedicated filetype for eslint. I don't think we need a separate "language" for emmet.

@NotAShelf should we split off conform/nvim-lint/lsp configuration from languages? the JS family has a fuck TON of duplication in their code (astro, svelte, ts, and possibly more)

github-actions bot pushed a commit that referenced this pull request Oct 15, 2025
@Bliztle Bliztle closed this Oct 30, 2025
@Bliztle Bliztle deleted the emmet-language-server branch October 30, 2025 16:00
@github-actions
Copy link

✅ Preview has been deleted successfully!

github-actions bot pushed a commit that referenced this pull request Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants