panoply / vscode-liquid

💧Liquid language support for VS Code
https://marketplace.visualstudio.com/items?itemName=sissel.shopify-liquid
Other
171 stars 22 forks source link

First attribute of long HTML tags is not indented #143

Closed sentience closed 10 months ago

sentience commented 1 year ago

This:

  <link rel="alternate" type="application/rss+xml" title="Kevin Yank posts feed" href="/feed.xml">

Gets formatted to this:

  <link
  rel="alternate"
    type="application/rss+xml"
    title="Kevin Yank posts feed"
    href="/feed.xml">

That un-indented rel attribute looks broken.

panoply commented 1 year ago

Hey @sentience

Thanks for reporting this. I believe that voids require self closing, eg:

  <link
    rel="alternate"
    type="application/rss+xml"
    title="Kevin Yank posts feed"
    href="/feed.xml" />

At least in this version that is the case. The next release fixes this issue.

panoply commented 10 months ago

This was fixed in v4.0.0 plus a who lot of different formatting issues. Thanks for submitting.