(HDS-2419) Remove useless native props

City-of-Helsinki · Oct 14, 2024 · 91d3ca6 · 91d3ca6
1 parent 26f141c
commit 91d3ca6
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 52 deletions.
diff --git a/packages/react/src/components/accordion/Accordion.tsx b/packages/react/src/components/accordion/Accordion.tsx
@@ -69,10 +69,6 @@ export type CommonAccordionProps = React.PropsWithChildren<{
    * @default 2
    */
   headingLevel?: number;
-  /**
-   * The id for the accordion element
-   */
-  id?: string;
   /**
    * Boolean indicating whether the accordion is initially opened.
    * @default false

diff --git a/packages/react/src/components/button/Button.tsx b/packages/react/src/components/button/Button.tsx
@@ -64,10 +64,6 @@ export type CommonButtonProps = AllElementPropsWithoutRef<'button'> & {
    * @default ButtonTheme.Bus
    */
   theme?: ButtonPresetTheme | ButtonTheme;
-  /**
-   * If `true`, the button will be disabled
-   */
-  disabled?: boolean;
   /**
    * If `true`, the button will take up the full width of its container
    */

diff --git a/packages/react/src/components/checkbox/Checkbox.tsx b/packages/react/src/components/checkbox/Checkbox.tsx
@@ -11,18 +11,10 @@ import { AllElementPropsWithoutRef, MergeAndOverrideProps } from '../../utils/el
 export type CheckboxProps = MergeAndOverrideProps<
   AllElementPropsWithoutRef<'input'>,
   {
-    /**
-     * If `true`, the component is checked
-     */
-    checked?: boolean;
     /**
      * Additional class names to apply to the checkbox
      */
     className?: string;
-    /**
-     * If `true`, the checkbox will be disabled
-     */
-    disabled?: boolean;
     /**
      * The error text content that will be shown below the checkbox
      */
@@ -43,18 +35,10 @@ export type CheckboxProps = MergeAndOverrideProps<
      * The label for the checkbox
      */
     label?: string | React.ReactNode;
-    /**
-     * Callback fired when the state is changed
-     */
-    onChange?: React.ChangeEventHandler<HTMLInputElement>;
     /**
      * Override or extend the styles applied to the component
      */
     style?: React.CSSProperties;
-    /**
-     * The value of the component
-     */
-    value?: string;
     /**
      * Tooltip text for the checkbox
      */

diff --git a/packages/react/src/components/radioButton/RadioButton.tsx b/packages/react/src/components/radioButton/RadioButton.tsx
@@ -8,18 +8,10 @@ import { AllElementPropsWithoutRef, MergeAndOverrideProps } from '../../utils/el
 export type RadioButtonProps = MergeAndOverrideProps<
   AllElementPropsWithoutRef<'input'>,
   {
-    /**
-     * If `true`, the component is checked
-     */
-    checked?: boolean;
     /**
      * Additional class names to apply to the radio button
      */
     className?: string;
-    /**
-     * If `true`, the radio button will be disabled
-     */
-    disabled?: boolean;
     /**
      * The helper text content that will be shown below the input
      */

diff --git a/packages/react/src/components/textInput/TextInput.tsx b/packages/react/src/components/textInput/TextInput.tsx
@@ -32,10 +32,6 @@ export type TextInputProps = MergeAndOverrideProps<
      * The default input element value. Use when the component is not controlled
      */
     defaultValue?: string;
-    /**
-     * If `true`, the input will be disabled
-     */
-    disabled?: boolean;
     /**
      * The error text content that will be shown below the input
      */
@@ -72,14 +68,6 @@ export type TextInputProps = MergeAndOverrideProps<
      * Short hint displayed in the input before the user enters a value
      */
     placeholder?: string;
-    /**
-     * If `true`, prevents the user from changing the value of the field (not from interacting with the field)
-     */
-    readOnly?: boolean;
-    /**
-     * If `true`, the label is displayed as required and the `input` element will be required
-     */
-    required?: boolean;
     /**
      * Override or extend the styles applied to the component
      */

diff --git a/packages/react/src/components/textarea/TextArea.tsx b/packages/react/src/components/textarea/TextArea.tsx
@@ -17,10 +17,6 @@ export type TextAreaProps = MergeAndOverrideProps<
      * The default textarea element value. Use when the component is not controlled
      */
     defaultValue?: string;
-    /**
-     * If `true`, the textarea will be disabled
-     */
-    disabled?: boolean;
     /**
      * The error text content that will be shown below the textarea
      */
@@ -53,10 +49,6 @@ export type TextAreaProps = MergeAndOverrideProps<
      * Short hint displayed in the textarea before the user enters a value
      */
     placeholder?: string;
-    /**
-     * If `true`, the label is displayed as required and the `textarea` element will be required
-     */
-    required?: boolean;
     /**
      * Override or extend the styles applied to the component. See text field [tokens](https://city-of-helsinki.github.io/helsinki-design-system/components/text-field#tokens) for available CSS variables
      */