I'd like to display my "discuss this" links only on the full nodes. There doesn't seem to be a way to do that now. Maybe that option can be added down the road?

In the meantime, if someone can suggest how I might code it, I'm open to a little improvising.

Comments

spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
commented

This is a reasonable feature to include. I'd suggest adding an option to the admin/settings/discussthis page (say, a dropdown select) so the admin can choose whether the links show up on Teaser Views, Full-node Views, or Both.

Using this variable, a slight modification to the hook_link implementation (and probably the hook_nodeapi one, too) should accomplish this feature easily.

I probably won't have time to code this in the next week or so, but if you feel confident rolling a patch, I'd happily review and commit it :)

aharown07’s picture

aharown07 commented

I'm afraid I'm too spotty in PHP & Drupal hooks to know how to do that. Mostly I cut and paste... and sometimes guess-edit-test-repeat (very slow process)! Would if I knew how though.

spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
commented
Assigned: Unassigned » spiderman
Status: Active » Fixed

Ok I've implemented this feature, although I'm not entirely happy with the UI. I've simply added a second "conditional" dropdown to the set of node types, which disappears when the checkbox is unchecked. My CSS skills are fairly rudimentary, and I couldn't work out a nice way to display them inline, and/or with a label indicating what they were for. Patches welcome ;)

commit is here: http://drupal.org/cvs?commit=168654

aharown07’s picture

aharown07 commented

Many thanks. Will try this out soon

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.