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(searchbar): remove unused Search prop showIcon #674

Merged
merged 2 commits into from
Jul 17, 2020
Merged

Conversation

jendowns
Copy link
Contributor

The showIcon prop passed to inner Search component is not a valid Search prop:
https://github.com/carbon-design-system/carbon/blob/f592f2d2540042befa0cc8e97cb31a0d9c52f748/packages/react/src/components/Search/Search.js#L18

Proposed changes

  • remove showIcon prop from Search component and update snapshots

@jendowns jendowns requested a review from a team as a code owner July 17, 2020 14:21
@jendowns jendowns requested a review from SimonFinney July 17, 2020 14:21
@netlify
Copy link

netlify bot commented Jul 17, 2020

Deploy preview for ibm-security ready!

Built with commit 3e2b4c4

https://deploy-preview-674--ibm-security.netlify.app

Copy link
Contributor

@SimonFinney SimonFinney left a comment

Choose a reason for hiding this comment

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

LGTM! Where did this even come from? 😆

@jendowns jendowns merged commit 6c5af0b into dev Jul 17, 2020
@jendowns jendowns deleted the searchbar-updates branch July 17, 2020 20:21
SimonFinney pushed a commit that referenced this pull request Jul 17, 2020
# [1.28.0-prerelease.3](v1.28.0-prerelease.2...v1.28.0-prerelease.3) (2020-07-17)

### Bug Fixes

* **searchbar:** remove unused Search prop showIcon ([#674](#674)) ([6c5af0b](6c5af0b))
@SimonFinney
Copy link
Contributor

🎉 This PR is included in version 1.28.0-prerelease.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

SimonFinney pushed a commit that referenced this pull request Jul 21, 2020
# [1.28.0](v1.27.0...v1.28.0) (2020-07-21)

### Bug Fixes

* **checkbox:** update checkbox focus to ensure border is used ([#672](#672)) ([53bd877](53bd877))
* **data-table:** proxy `defaultProps` and `propTypes` ([#666](#666)) ([cf37860](cf37860))
* **decorator:** add character limit, display value in `title` prop ([#654](#654)) ([0adbcfc](0adbcfc))
* **dropdown:** add `ListBox` styles ([#659](#659)) ([644c389](644c389))
* **searchbar:** remove unused Search prop showIcon ([#674](#674)) ([6c5af0b](6c5af0b))
* **truncated-list:** include `ScrollGradient` styles ([#676](#676)) ([1c44825](1c44825))

### Features

* **carbon:** upgrade to `v10.15` ([#660](#660)) ([f328809](f328809))
@SimonFinney
Copy link
Contributor

🎉 This PR is included in version 1.28.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants