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(masthead-search): no redirect on empty value #3932

Merged
merged 3 commits into from
Sep 14, 2020

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented Sep 14, 2020

Related Ticket(s)

Refs #3883.

Description

This change ensures redirect to dedicated search page upon clicking on
the loupe icon happens only when there is a search query in the input
box.

Changelog

Changed

  • A change to ensure redirect to dedicated search page upon clicking on
    the loupe icon happens only when there is a search query in the input
    box.

This change ensures redirect to dedicated search page upon clicking on
the loupe icon happens only when there is a search query in the input
box.

Refs carbon-design-system#3883.
@asudoh asudoh added the package: web components Work necessary for the IBM.com Library web components package label Sep 14, 2020
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Sep 14, 2020

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Sep 14, 2020

Copy link
Member

@jeffchew jeffchew left a comment

Choose a reason for hiding this comment

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

Thank you, good add!

@jeffchew jeffchew added the Ready to merge Label for the pull requests that are ready to merge label Sep 14, 2020
kodiakhq bot pushed a commit that referenced this pull request Sep 14, 2020
### Related Ticket(s)

React version fix of #3932

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "package: vanilla": Vanilla -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "package: styles": Carbon Expressive, React (Expressive) -->
<!-- *** "RTL": React (RTL) -->
<!-- *** "feature flag": React (experimental) -->
@kodiakhq kodiakhq bot merged commit 89f4553 into carbon-design-system:master Sep 14, 2020
@asudoh asudoh deleted the search-redirect branch September 14, 2020 21:39
asudoh added a commit to asudoh/ibm-dotcom-library that referenced this pull request Oct 14, 2020
kodiakhq bot pushed a commit that referenced this pull request Oct 14, 2020
### Related Ticket(s)

Refs #3883.

### Description

Re-applies #3932 as #3933 ended up with erasing #3932.

### Changelog

**Changed**

- Re-applies #3932 as #3933 ended up with erasing #3932.
ariellalgilmore pushed a commit to ariellalgilmore/carbon-for-ibm-dotcom that referenced this pull request Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: web components Work necessary for the IBM.com Library web components package Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants