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 Destroy Method #1396

Merged
merged 10 commits into from
Jul 26, 2013
Merged

Add Destroy Method #1396

merged 10 commits into from
Jul 26, 2013

Conversation

pfiller
Copy link
Contributor

@pfiller pfiller commented Jul 24, 2013

@harvesthq/chosen-developers

This brings @koenpunt's destroy feature into the Chosen repo (albeit on a branch: koenpunt-chosen-destory). As far as I can tell, there's only one thing left to do:

  • Make sure events listeners aren't leaking

Closes #133, #1127 and #505

@pfiller
Copy link
Contributor Author

pfiller commented Jul 24, 2013

I didn't reference #1379 in the opening comment. Now I have!

@pfiller
Copy link
Contributor Author

pfiller commented Jul 24, 2013

Listeners look good now. What say you guys?

screen shot 2013-07-24 at 4 24 06 pm

@stof
Copy link
Collaborator

stof commented Jul 24, 2013

is it also good in the jquery version ?

@pfiller
Copy link
Contributor Author

pfiller commented Jul 24, 2013

is it also good in the jquery version ?

Yes, jQuery's remove seems to be doing its job and all child event listeners are cleared up appropriately.

Conflicts:
	coffee/chosen.jquery.coffee
pfiller added a commit that referenced this pull request Jul 26, 2013
@pfiller pfiller merged commit 5d22658 into master Jul 26, 2013
@pfiller pfiller deleted the koenpunt-chosen-destroy branch July 26, 2013 14:34
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.

There is no way to remove chosen from the page
3 participants