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

Add an 'X' button for clearing text in the search field. #62

Merged
merged 1 commit into from
Jan 28, 2014

Conversation

birdwing
Copy link
Contributor

I saw on issue #35 you needed a way to clear the search box.
Here is a simple edit. Of the development files

unfortunately I am unable to run the bash script to minimize the css and js, but since I didn't add any files you should have no problem with it.

I hope this helps :)

@@ -122,10 +122,31 @@ header {
-moz-transition: opacity .3s ease-out, -moz-transform .3s ease-out, box-shadow .3s, width .2s ease-out;
transition: opacity .3s ease-out, transform .3s ease-out, box-shadow .3s, width .2s ease-out;
}
header #clearSearch {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

CSS Code for the "X" or Clear Search button.
It had to be position: absolute due to the resizing of the search box.

@electerious electerious merged commit 5f61b1b into electerious:master Jan 28, 2014
@electerious
Copy link
Owner

Hi @birdwing, good work! :) I made some small enhancements and it works great!

@birdwing
Copy link
Contributor Author

Thanks! Glad I could help!

@birdwing
Copy link
Contributor Author

I took a look at the changes, I guess I should have noticed you had a reset function already, and i love the use of the CSS transitions instead of the :hover I was using. I really should look more into those.

Awesome project!

@birdwing birdwing deleted the ClearSearch branch January 28, 2014 18:02
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.

2 participants