harvesthq/chosen — pull request #1237 by pfiller (2013-05-24 19:32:17+00:00)
Title: Another Take on Disabled Options

@kenearley I kept staring at #1177 and hating all those tests for ``disabled-result``. We're already testing for ``active-result`` and it all seemed very messy to me.

When I was getting ready to merge, I had an idea. If we make disabled results show up without needing the active-result class, we can remove all those tests. So I tried it and it seems to work pretty good.

The only problem I've found so far (and it's not a small one) is that disabled results aren't disappearing when they should (on a search that doesn't match). It's tricky, but I think it's workaroundable (NEW WORD).

What do you think of this approach?

**Note:** The target for this merge is ``disabled-options-take-2``, not ``master``