Migrate Checkbox story to tsx - see #18100 (#18101)

Looks good!
apache · Jan 20, 2022 · c3a17a7 · c3a17a7
1 parent 5ce79ff
commit c3a17a7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/.../components/Checkbox/Checkbox.stories.jsx → .../components/Checkbox/Checkbox.stories.tsx b/.../components/Checkbox/Checkbox.stories.jsx → .../components/Checkbox/Checkbox.stories.tsx
@@ -18,7 +18,7 @@
  */
 import React from 'react';
 import { useArgs } from '@storybook/client-api';
-import Checkbox from '.';
+import Checkbox, { CheckboxProps } from '.';
 
 export default {
   title: 'Checkbox',
@@ -42,9 +42,8 @@ export const CheckboxGallery = () =>
     </div>
   ));
 
-// eslint-disable-next-line no-unused-vars
-export const InteractiveCheckbox = _args => {
-  const [{ checked, style }, updateArgs] = useArgs();
+export const InteractiveCheckbox = ({ checked, style }: CheckboxProps) => {
+  const [, updateArgs] = useArgs();
   const toggleCheckbox = () => {
     updateArgs({ checked: !checked });
   };

diff --git a/superset-frontend/src/components/Checkbox/Checkbox.tsx b/superset-frontend/src/components/Checkbox/Checkbox.tsx
@@ -49,3 +49,5 @@ export default function Checkbox({ checked, onChange, style }: CheckboxProps) {
     </Styles>
   );
 }
+
+export type { CheckboxProps };
diff --git a/superset-frontend/src/components/Checkbox/index.tsx b/superset-frontend/src/components/Checkbox/index.tsx
@@ -16,5 +16,5 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-export { default } from 'src/components/Checkbox/Checkbox';
+export { default, CheckboxProps } from 'src/components/Checkbox/Checkbox';
 export * from 'src/components/Checkbox/CheckboxIcons';