You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should refactor IInputGroupProps to extend from HTMLInputProps, so that users can add custom callbacks on focus, on blur, and so on. The problem at present is that IInputGroupProps already extends from IControlledProps, which exposes props that conflict with some of those also included in HTMLInputProps (in particular: onChange, defaultValue, and value). Here's the exact set of errors you see:
[ts]
Interface 'IInputGroupProps' cannot simultaneously extend types 'IControlledProps' and 'HTMLProps<HTMLInputElement>'.
Named property 'defaultValue' of types 'IControlledProps' and 'HTMLProps<HTMLInputElement>'
are not identical.
[ts]
Interface 'IInputGroupProps' cannot simultaneously extend types 'IControlledProps' and 'HTMLProps<HTMLInputElement>'.
Named property 'onChange' of types 'IControlledProps' and 'HTMLProps<HTMLInputElement>' are
not identical.
[ts]
Interface 'IInputGroupProps' cannot simultaneously extend types 'IControlledProps' and 'HTMLProps<HTMLInputElement>'.
Named property 'value' of types 'IControlledProps' and 'HTMLProps<HTMLInputElement>' are not
identical.
EDIT: There may be a better way to achieve this; the goal is primarily to expose the HTML Input lifecycle callbacks somehow.
The text was updated successfully, but these errors were encountered:
in the React types, we use HTMLProps, but there's also HTMLAttributes (HTMLProps minus the React attributes). We should probably always use HTMLAttributes
We should refactor
IInputGroupProps
to extend fromHTMLInputProps
, so that users can add custom callbacks on focus, on blur, and so on. The problem at present is thatIInputGroupProps
already extends fromIControlledProps
, which exposes props that conflict with some of those also included inHTMLInputProps
(in particular:onChange
,defaultValue
, andvalue
). Here's the exact set of errors you see:EDIT: There may be a better way to achieve this; the goal is primarily to expose the HTML Input lifecycle callbacks somehow.
The text was updated successfully, but these errors were encountered: