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(search-box): back arrow clears search (closes #1157) #1158

Merged
merged 3 commits into from
May 28, 2018

Conversation

JulieMarie
Copy link
Collaborator

@JulieMarie JulieMarie commented May 24, 2018

Description

Clear value in search box when back arrow is tapped
Fixes bug found in issue: Covalent-1157

Test Steps

  • checkout branch
  • npm run serve
  • Go to Components -> Data Table -> Data Table with components
  • Enter search text in search field
  • Verify corresponding results shown
  • Click on the back arrow in search field
  • Verify initial table is displayed

General Tests for Every PR

  • npm run serve:prod still works.
  • npm run tslint passes.
  • npm run stylelint passes.
  • npm test passes and code coverage is not lower.
  • npm run build:lib still works.
Screenshots or link to StackBlitz/Plunker

covalent-search

closes #1157

@emoralesb05 emoralesb05 changed the title fix(search-box): back arrow clears search fix(search-box): back arrow clears search (closes #1157) May 25, 2018
@emoralesb05 emoralesb05 merged commit 4aa601f into develop May 28, 2018
@emoralesb05 emoralesb05 deleted the fix/back-arrow-clear-search branch May 28, 2018 18:40
emoralesb05 pushed a commit that referenced this pull request May 28, 2018
* fix(search-box): back arrow clears search

* fix(seach-box): actually clear the input
kriswinbush pushed a commit to kriswinbush/covalent that referenced this pull request Feb 20, 2020
…adata#1158)

* fix(search-box): back arrow clears search

* fix(seach-box): actually clear the input
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.

Data Table with components: search result wont get cleared after clicking back arrow
2 participants