fix: fix property transform

alibaba · Sep 5, 2024 · 4626528 · 4626528
1 parent 97d82df
commit 4626528
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/packages/rax-compat/src/props.ts b/packages/rax-compat/src/props.ts
@@ -27,16 +27,16 @@ function transformProps(type: string, props: ComponentProps<JSXElementConstructo
         }
       }
       // eslint-disable-next-line no-prototype-builtins
-    }
-
-    const needTransform = (type !== 'svg');
-    if (needTransform) {
-      if (Object.prototype.hasOwnProperty.call(possibleStandardNames, lowerCasedPropKey)) {
-        // Transform attribute names that make it works properly in React.
-        key = possibleStandardNames[lowerCasedPropKey];
-      } else {
-        // Handles component props from rax-components like resizeMode, this causes React to throw a warning.
-        key = lowerCasedPropKey;
+    } else {
+      const needTransform = (type !== 'svg');
+      if (needTransform) {
+        if (Object.prototype.hasOwnProperty.call(possibleStandardNames, lowerCasedPropKey)) {
+          // Transform attribute names that make it works properly in React.
+          key = possibleStandardNames[lowerCasedPropKey];
+        } else {
+          // Handles component props from rax-components like resizeMode, this causes React to throw a warning.
+          key = lowerCasedPropKey;
+        }
       }
     }
 

diff --git a/packages/rax-compat/tests/events.test.tsx b/packages/rax-compat/tests/events.test.tsx
@@ -58,6 +58,9 @@ describe('events', () => {
   });
 
   it('should work with ondblclick', () => {
+    console.log('aaaaaa', transformProps('div', {
+      ondblclick: () => { },
+    }));
     expect(transformProps('div', {
       ondblclick: () => { },
     }).onDoubleClick).toBeInstanceOf(Function);