Hi
Thank you for a great module.
I've made some minor changes to the javascript file, so terms can be both added and removed using the clickable list.
I've also added a 'selected' class to any selected terms, to help theming this module.
I sorry but I don't know how to create a patch, so I attached the modifed javascript file instead. Hope that's ok.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 206896-remove-enhanced-8.patch | 4.98 KB | bbinkovitz |
| #7 | remove_enhanced.patch | 5.02 KB | Steve Dondley |
| #6 | remove_terms.patch | 1.91 KB | Steve Dondley |
| #3 | suggestedterms.js_.patch | 2.18 KB | thulstrup |
| suggestedterms.js_.txt | 1.34 KB | thulstrup |
Comments
Comment #1
Crell commentedD'Oh! And right after I release a 1.1 stable, too. :-) Sounds like a nice feature.
Please create a proper patch so that other people can review it properly. See: http://drupal.org/patch/create
Thanks.
Comment #2
thulstrup commentedThanks, I'll give it a try.
Comment #3
thulstrup commentedOk, here's the patch.
This is my first patch ever, so I hope everything is as it's supposed to be :-)
Comment #4
Crell commentedSo I'm finally getting back to this, and the patch doesn't apply. :-( Sorry. Also, please make sure you use Unix line endings, not Windows line endings. Thanks.
Comment #5
Crell commentedNew features are only being accepted in the 6.x branch now.
Comment #6
Steve Dondley commentedHere's another patch for removing terms. It is completely new and does not build upon the work of the previous contributor in this thread.
Comment #7
Steve Dondley commentedHere's an "enhanced" patch that adds some logic and css to make terms that are selected to display as red hyperlinks. Terms go back to the default link color when the term is removed.
Comment #8
bbinkovitz commentedI was not able to apply the patch from #6. I did apply #7 and it seems to work nicely. I have re-rolled it to fix a minor whitespace problem.
Comment #9
bbinkovitz commentedThis has been committed to the latest dev.