Hi ,
I have been improving this module as part of a project and would be interested with maintenance, as the status is looking for co-maintainer.
Please find the patch attached, it solves the following tickets:
- #1114510: Nested taxonomy lists collapse unintentionally (using an identical approach of the proposed patch)
- #789712: Enable option to force selection of parents terms
- other non reported small issues were addressed
Please review the patch and get back to me about the maintenance. I'll have more improvements coming up, that will add extra functionality to it.
Please note that these improvements were sponsored; it is alway appreciated the use of --author in patch commits :-)
Thanks!
Nuno
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | taxonomy_tree_select-1351972.patch | 9.33 KB | nunoveloso |
Comments
Comment #1
nunoveloso commentedAnd now the patch file :-)
Comment #2
danillonunes commentedHello Nuno,
Please do not comment a piece of code. If you want to remove something, just remove it.
I don't know if we should rely on this property or leave it to theme layer.
Extra whitespace.
I think this needs a server-side validation too.
This was already commited, but I liked the more descriptive 'event' variable name.
Extra whitespace.
--
Please send a separate patch in #789712: Enable option to force selection of parents terms with only their related changes. Also, if you want, submit a follow-up patch in #1114510: Nested taxonomy lists collapse unintentionally with the event variable name changes.
Thank you.