feat: noStyle no warning (#48508)

ant-design · Apr 17, 2024 · 0d34e0b · 0d34e0b
1 parent a512f81
commit 0d34e0b
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/components/form/FormItem/index.tsx b/components/form/FormItem/index.tsx
@@ -427,7 +427,7 @@ function InternalFormItem<Values = any>(props: FormItemProps<Values>): React.Rea
           childNode = mergedChildren(context as any);
         } else {
           warning(
-            !mergedName.length,
+            !mergedName.length || !!noStyle,
             'usage',
             '`name` is only used for validate React element. If you are using Form.Item as layout display, please remove `name` instead.',
           );

diff --git a/components/form/__tests__/index.test.tsx b/components/form/__tests__/index.test.tsx
@@ -497,6 +497,15 @@ describe('Form', () => {
     );
   });
 
+  it('No warning when use noStyle and children is empty', () => {
+    render(
+      <Form>
+        <Form.Item name="noWarning" noStyle />
+      </Form>,
+    );
+    expect(errorSpy).not.toHaveBeenCalled();
+  });
+
   it('dynamic change required', async () => {
     const { container } = render(
       <Form>