diff --git a/packages/react/src/components/Image/Image.tsx b/packages/react/src/components/Image/Image.tsx index 6ac0838e..46032ee8 100644 --- a/packages/react/src/components/Image/Image.tsx +++ b/packages/react/src/components/Image/Image.tsx @@ -25,6 +25,10 @@ export type ImageProps = ImgHTMLAttributes; const COMPONENT_NAME: string = 'Image'; +/** + * TODO: Refer improvement issue if this Image component is required. + * @see {@link https://github.com/wso2/oxygen-ui/issues/65} + */ const Image: FC & WithWrapperProps = (props: ImageProps): ReactElement => { const {className, alt, ...rest} = props; diff --git a/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.stories.mdx b/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.stories.mdx index 66720b56..6c0af927 100644 --- a/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.stories.mdx +++ b/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.stories.mdx @@ -43,6 +43,18 @@ Import and use the `PhoneNumberInput` component in your components as follows. code={dedent` import PhoneNumberInput from '@oxygen-ui/react/PhoneNumberInput';\n function Demo() { - return ; + const [dialCode, setDialCode] = useState(''); + const [phoneNumber, setPhoneNumber] = useState('');\n + const handlePhoneNumberInputChange = (dialCode, phoneNumber) => { + setDialCode(dialCode); + setPhoneNumber(phoneNumber); + };\n + return ( + + ); }`} /> diff --git a/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.tsx b/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.tsx index 43231599..19d63bc6 100644 --- a/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.tsx +++ b/packages/react/src/components/PhoneNumberInput/PhoneNumberInput.tsx @@ -46,15 +46,21 @@ export interface PhoneNumberInputProps extends BoxProps { */ SelectProps?: Omit; /** - * Default country selected for the dialCode. + * Dial code state value. * - * @example {code: 'US', dialCode: '+1', name: 'United States'} + * @example '+94' */ - defaultCountry?: Country; + dialCodeValue?: string; /** * Callback function to be called when the dialCode or phoneNumber changes. */ onChange?: (dialCode: string, phoneNumber: string) => void; + /** + * Phone number state value. + * + * @example '787878787' + */ + phoneNumberValue?: string; /** * Placeholder text for the phone number input. */ @@ -67,11 +73,12 @@ const PhoneNumberInput: ForwardRefExoticComponent & WithW (props: PhoneNumberInputProps, ref: MutableRefObject): ReactElement => { const { className, - defaultCountry, + dialCodeValue, label, InputLabelProps, OutlinedInputProps, onChange, + phoneNumberValue, placeholder, SelectProps, ...rest @@ -79,17 +86,36 @@ const PhoneNumberInput: ForwardRefExoticComponent & WithW const classes: string = clsx('oxygen-phone-number-input', className); - const [country, setCountry] = useState(defaultCountry ?? countries[0]); - const [phoneNumber, setPhoneNumber] = useState(''); + const [dialCode, setDialCode] = useState(dialCodeValue ?? countries[0].dialCode); + + const [phoneNumber, setPhoneNumber] = useState(phoneNumberValue ?? ''); const handleDialCodeChange = (event: SelectChangeEvent): void => { - setCountry(countries.find((item: Country) => item.dialCode === event.target.value)); + setDialCode(event.target.value); onChange?.(event.target.value, phoneNumber); }; const handlePhoneNumberChange = (event: ChangeEvent): void => { setPhoneNumber(event.target.value); - onChange?.(country.dialCode, event.target.value); + onChange?.(dialCode, event.target.value); + }; + + const renderValue = (value: string): ReactElement => { + const selectedCountry: Country = countries.find((item: Country) => item.dialCode === dialCode); + + return ( + <> + + } + /> + + {value} + + ); }; return ( @@ -102,30 +128,23 @@ const PhoneNumberInput: ForwardRefExoticComponent & WithW className="oxygen-select" labelId="phone-number-label" id="phone-number-select" - value={country.dialCode} + value={dialCode} onChange={handleDialCodeChange} - renderValue={(value: string): ReactElement => ( - <> - - } /> - - {value} - - )} + renderValue={renderValue} inputProps={{ className: 'oxygen-select-input-root', }} {...SelectProps} > {countries?.map((countryItem: Country) => { - const {dialCode, code, name} = countryItem; + const {dialCode: phoneCode, code, name} = countryItem; return ( - + - } /> + } /> {name}  - {dialCode} + {phoneCode} ); })}