Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(searchinput): weird type bug resolved #1698

Merged
merged 3 commits into from
Apr 2, 2021
Merged

Conversation

EdwardIrby
Copy link
Contributor

That was a weird one. I ended up playing with it until I got the build to output something more inline with that TextInput was outputting as it was doing a weird extension of it. Should resolve the issue I hope.

Resolves: #1683

@EdwardIrby EdwardIrby merged commit 3845537 into master Apr 2, 2021
@EdwardIrby EdwardIrby deleted the fix/searchinput-types branch April 2, 2021 04:25
jaketrent added a commit that referenced this pull request Apr 7, 2021
* master:
  build: publish
  fix(datepicker): closes on click outside resolves #1655 (#1700)
  tagsinput: fix scroll bar position when items/pills overflow  (#1702)
  flexibility: remove base from color scale & add parseToRgb (#1696)
  fix(searchinput): weird type bug resolved (#1698)
  feat(core): temp core util for refactoring to vanillia css (#1689)
  build: publish
  select: stop intercepting all key events (#1699)
  build: publish
  fix(storybook-addon-theme): update incorrect package name in README
  feat(storybook-addon-theme): update icon to circle
  feat(storybook-addon-theme): ability to set theme per-story
  util: polymorphic primatives (#1690)
  build: publish
  fix(select): dropdown zindex added (#1691)
  Update @pluralsight/ps-design-system-storybook-addon-theme package.json for addon-catalog
  Update @pluralsight/ps-design-system-storybook-addon-center package.json for addon-catalog
@EdwardIrby EdwardIrby mentioned this pull request Jun 9, 2021
Comment on lines -21 to +23
export interface SearchInputProps
extends React.ComponentProps<typeof TextInput> {
export interface SearchInputProps extends TextInputProps {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jaketrent this the example I'm talking about

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[searchinput]: typescript requires a css prop
2 participants