fix(PropsProxy): only convert jsx propnames for string nodes

Author: wmertens

packages/qwik/src/core/shared/jsx/props-proxy.ts

@@ -28,7 +28,7 @@ class PropsProxyHandler implements ProxyHandler<any> {
     if (prop === 'children') {
       value = this.owner.children;
     } else {
-      if (typeof prop === 'string') {
+      if (typeof prop === 'string' && typeof this.owner.type === 'string') {
         const attr = jsxEventToHtmlAttribute(prop as string);
         if (attr) {
           prop = attr;
@@ -48,7 +48,7 @@ class PropsProxyHandler implements ProxyHandler<any> {
     } else if (prop === 'children') {
       this.owner.children = value;
     } else {
-      if (typeof prop === 'string') {
+      if (typeof prop === 'string' && typeof this.owner.type === 'string') {
         const attr = jsxEventToHtmlAttribute(prop as string);
         if (attr) {
           prop = attr;
@@ -90,7 +90,7 @@ class PropsProxyHandler implements ProxyHandler<any> {
     } else if (prop === _CONST_PROPS || prop === _VAR_PROPS) {
       return true;
     }
-    if (typeof prop === 'string') {
+    if (typeof prop === 'string' && typeof this.owner.type === 'string') {
       const attr = jsxEventToHtmlAttribute(prop as string);
       if (attr) {
         prop = attr;
@@ -133,6 +133,8 @@ class PropsProxyHandler implements ProxyHandler<any> {
 /**
  * Instead of using PropsProxyHandler getter (which could create a component-level subscription).
  * Use this function to get the props directly from a const or var props.
+ *
+ * This does not convert jsx event names.
  */
 export const directGetPropsProxyProp = <T, JSX>(jsx: JSXNodeInternal<JSX>, prop: string): T => {
   return (