The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi,
I configured this module with success, great to see it's simple and effective, and usefull !
However, I would like to theme it in a complete different way because I find using table here is not very semantically correct and I found that a lot of the page is generated in a non theme function.
I could easily remove some div, hide some description etc... but the main table is not modifiable without hacking core.
It is possible to correct that ?
I will continue to look through the code
thanks
zmove
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | taxonomy_list.module.patch | 49.16 KB | zmove |
| #1 | taxonomy_list.module.patch | 49.16 KB | zmove |
Comments
Comment #1
zmove commentedPretty Easy to do, this is a patch that make _taxonomy_list_get_table themable. it allow the user to create something else than a table if he want.
I didn't touch it, but maybe if the user can do another thing than a table, maybe renaming the function is a good idea ;)
What's your opinion about this patch ?
Comment #2
zmove commenteddouble post
Comment #3
zmove commentedPretty Easy to do, this is a patch that make _taxonomy_list_get_table themable. it allow the user to create something else than a table if he want.
I didn't touch it, but maybe if the user can do another thing than a table, maybe renaming the function is a good idea ;)
What's your opinion about this patch ?
Comment #4
nancydruThis is not really a patch, but a complete replacement. In order to make a "proper" patch, please use "diff" with the options "-u -p". It will be a much smaller file.
Comment #5
zmove commentedHmm, I'm on winerge and I follow the handbook about it. I don't know where yo add "-u -p" on windows winmerge.
Comment #6
nancydruActually, I've never figured out how to get it to do the "-p" but the "-u" seems to happen automatically when it's called by CVS.
You can download a package called "diffutils" (http://sourceforge.net/project/shownotes.php?release_id=83227). It's not quite as easy to use as CVS/Winmerge, but it's not hard.
Comment #7
zmove commentedOk thank you, I will take a look into it !
Comment #8
nancydruFix committed on 6.x-1.x-dev only. Note that after downloading the code, you must disable and re-enable the module for the new theme function to be recognized.
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.