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

Always clear result highlight #1407

Merged
merged 1 commit into from
Jul 26, 2013
Merged

Always clear result highlight #1407

merged 1 commit into from
Jul 26, 2013

Conversation

koenpunt
Copy link
Collaborator

Fixes #1406

@koenpunt
Copy link
Collaborator Author

I think we now also can remove the call to result_clear_highlight on some other places.

@kenearley
Copy link

Looks good. :shipit:

pfiller added a commit that referenced this pull request Jul 26, 2013
Always clear result highlight
@pfiller pfiller merged commit 77efc4f into harvesthq:master Jul 26, 2013
@pfiller
Copy link
Contributor

pfiller commented Jul 26, 2013

This function is actually prime for a refactor, but I'm cool with this stop gap.

@koenpunt koenpunt deleted the gh-1406 branch July 26, 2013 14:35
pfiller added a commit that referenced this pull request Jul 26, 2013
- CSS Updates #1368, #1371
- Prevent text selection on Chosen #1374
- single_backstroke_delete defaults to true #1375
- No longer expose classes to window context #1389
- Single select value fix #1392
- Remove javascript:void and attr references #1385, #1377
- Disable Chosen on iPhone / iPod and Android mobile devices
  by default #1388
- Add Destroy Method #1396
- Option to Hide Selected & Disabled Options #1404
- Always clear result highlight #1407
- Replace characters only once #1411

- Document all the things. #1366, #1376, #1383
- Add Gemfile for compass dependency #1373
- Update copyright and license link #1397
- Add @koenpunt as a maintainer #1399
- Add anchors to documentation headers #1395
- Updated stackoverflow instructions #1403
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.

Pressing enter will select the wrong option in certain cases
3 participants