Closed (fixed)
Project:
Ordered Taxonomy Display - Taxi
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
9 May 2008 at 16:14 UTC
Updated:
30 May 2008 at 03:43 UTC
Thanks for the great module - it came at just about the most perfect time for me. I've rolled a patch that allows the admin to set the weight for the display of the taxi module output. Let me know what you think.
| Comment | File | Size | Author |
|---|---|---|---|
| taxi_weight.patch | 1017 bytes | jrbeeman |
Comments
Comment #1
LisandroEC commentedGreat, I'll add the patch to the code, it sure is useful. Thanks a lot! And it's great you've found the module useful.
As it is already obvious, I'm not trained in this cvs/project page workflow, so here's a question, what should I do? Should I create a new release increasing minor version number, or add the code to a development tag? What are the usual guidelines on this?
Comment #2
jrbeemanIt's sort of your personal preference on how you'd like to manage the releases. I tend to commit everything to a development branch and then just tag releases along the way. If you feel like this change is worth a new release, then make a new tag - otherwise, just wait until some further enhancements or fixes are made to make an actual release tag.
Comment #3
asb commentedHi,
I fully agree with jrbeeman, this modules comes in quite handy to get some order in displaying taxonomy terms.
Is the customizable display weight patch already in the 5.x-1.x-dev release?
@jrbeeman: The patch applies cleanly and does work fine, thank you!
However, without theming, nodes with images attached by image_attach submodule are looking a bit weird (with the same weight, the output from "Taxi" is displayed to the left of the image, leaving no room for the node's body; I couldn't get the "Taxi" box to output exactly below the image; maybe I'd get better results with the CCK imagefield and some theming).
What I'd really love to see is the possibility to set Display weights per Vocabulary or even per content type.
@LisandroEC: What are your plans for the "Taxi" module? Will you port it to D6? Are you planning to add more features (e.g. option to exclude the display of hierarchical top terms), or is this more a module provided "as is"?
Thank you for this module & greetings, -asb
Comment #4
LisandroEC commentedThanks all for the comments and the feedback.
As soon as I have a little time to spare (I'm a bit cluttered at work, sorry!), I'll add the patch to the code and release an update.
@asb: Yes, I hope I'll have time to port it to D6, as well as adding new features to taxi. Please, could you describe a little more what features would you like implemented?
Comment #5
asb commentedHi LisandroEC,
since I don't want to hijack this issue: I'll create a wishlist in a separate feature request if I had some time to get used to "Taxi".
Thanks & greetings from Germany, -asb
Comment #6
LisandroEC commentedPatch added to 5.x-1.2.