Customizable titles everywhere using tokens
Todd Nienkerk - June 30, 2009 - 21:37
| Project: | Talk |
| Version: | 6.x-1.x-dev |
| Component: | User interface |
| Category: | feature request |
| Priority: | normal |
| Assigned: | greggles |
| Status: | closed |
Description
It would be useful to allow customization of the talk page's title in the UI. In order to allow the node title to appear in this custom title, tokens would need to be enabled.
Currently, the page title is hard-coded. Here's 6.x-1.5:
drupal_set_title(check_plain($node->title));And 6.x-1.x-dev as of June 25, 2009:
drupal_set_title(t('@title page for @node', array('@title' => talk_title(), '@node' => $node->title)));
#1
#2
And a patch...to provide this.
I actually decided to provide 3 different settings:
1. For the title of the talk page
2. For the "link" on the "links" area of a node which can be left blank to disable it
3. For the Tab that shows up next to "View" and "Edit"
I created an update function that creates the three values and defaults them to the value from the one that was previously customizable. It also gives a hint to the user that they should configure further.
#3
Coding standards, * Optional node object. needs to be further indented.
#4
I hope to take a look at this patch soon! will let you know what i think of the functionality when i have.
#5
Here's a re-rolled patch.
@GregoryHeller, that would be great. I've also posted a screenshot.
#6
I also committed #511026: provide token for number of comments on a node to token module which is really useful in this area... I can roll a token release to coincide with the next talk release so we have good support here.
#7
And a re-roll after #517988: Three bugs with "Add new comment" link when comment form is set to "display on separate page".
#8
Committed to the 6.x-1.x branch: http://drupal.org/cvs?commit=238066
I also provided more guidance in the README.txt for the way to use this feature.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.