(fix) svg attributes and casing

packages/language-server/src/plugins/PluginHost.ts

@@ -141,7 +141,9 @@ export class PluginHost implements LSProvider, OnWatchFileChanges {
                     return false;
                 }
                 if (label[0] === '"' && label[label.length - 1] === '"') {
-                    if (htmlCompletions.has(label.slice(1, -1))) {
+                    // this will result in a wrong completion regardless, remove the quotes
+                    item.label = item.label.slice(1, -1);
+                    if (htmlCompletions.has(item.label)) {
                         // "aria-label" -> aria-label -> exists in html completions
                         return false;
                     }

packages/language-server/src/plugins/typescript/features/SemanticTokensProvider.ts

@@ -77,7 +77,7 @@ export class SemanticTokensProviderImpl implements SemanticTokensProvider {
                 continue;
             }
 
-            const [line, character, length, start] = originalPosition;
+            const [line, character, length] = originalPosition;
 
             // remove identifiers whose start and end mapped to the same location,
             // like the svelte2tsx inserted render function,

packages/svelte2tsx/src/htmlxtojsx_v2/nodes/Attribute.ts

@@ -111,7 +111,7 @@ export function handleAttribute(
         return;
     } else {
         let name =
-            element instanceof Element && attr.value === true
+            element instanceof Element && parent.type === 'Element'
                 ? transformAttributeCase(attr.name)
                 : attr.name;
         // surround with quotes because dashes or other invalid property characters could be part of the name

packages/svelte2tsx/svelte-jsx.d.ts

@@ -68,7 +68,10 @@ declare namespace svelteHTML {
   interface AriaAttributes extends svelte.JSX.AriaAttributes {}
 
   // eslint-disable-next-line @typescript-eslint/no-empty-interface
-  interface HTMLAttributes<T extends EventTarget> extends svelte.JSX.HTMLAttributes<T> {
+  interface HTMLAttributes<T extends EventTarget> extends svelte.JSX.HTMLAttributes<T> {}
+
+  // eslint-disable-next-line @typescript-eslint/no-empty-interface
+  interface SVGAttributes<T extends EventTarget> extends svelte.JSX.SVGAttributes<T> {
     'xlink:actuate'?: string | undefined;
     'xlink:arcrole'?: string | undefined;
     'xlink:href'?: string | undefined;
@@ -83,9 +86,6 @@ declare namespace svelteHTML {
     'xmlns:xlink'?: string | undefined;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-empty-interface
-  interface SVGAttributes<T extends EventTarget> extends svelte.JSX.SVGAttributes<T> {}
-
   // eslint-disable-next-line @typescript-eslint/no-empty-interface
   interface HTMLProps<T extends EventTarget> extends HTMLAttributes<T> {}
 

packages/svelte2tsx/test/htmlx2jsx/samples/attribute-element/expectedv2.js

@@ -1,3 +1,3 @@
  { svelteHTML.createElement("div", {"contenteditable":true,}); }
  { svelteHTML.createElement("div", { contentEditable,}); }
- { svelteHTML.createElement("div", { "contentEditable":contenteditable,}); }
+ { svelteHTML.createElement("div", { "contenteditable":contenteditable,}); }