Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
You have no options for the title of the sitemap page. It can only say "Site map", be in the one spot, and can't have any css applied to it.
Comment | File | Size | Author |
---|---|---|---|
#10 | site_map_730992-10.patch | 1.75 KB | tic2000 |
#7 | site_map_730992-7.patch | 1.74 KB | tic2000 |
Comments
Comment #1
hass CreditAttribution: hass commented#247077: Add (node/block)-site_map.tpl.php support for custom theming
Comment #2
tic2000 CreditAttribution: tic2000 commentedThis is not a duplicate.
Every page in Drupal is like this. You can create a page-sitemap.tpl.php in your theme folder and change the default style and title.
The only solution would be to add a title field in the settings page and use that title, but that won't change the style of it which is set in page.tpl.php and has nothing to do with this or any other module.
Comment #3
hass CreditAttribution: hass commentedNo. All you said is wrong.
He is asking for adding a css class to the title and this will be possible with the new template
Comment #4
tic2000 CreditAttribution: tic2000 commentedStop arguing if you have no idea what you're talking about.
The "Site map" title is not printed in any function or file in site_map module. The title is just set in site_map_menu().
Like any other title, it's on page.tpl.php and to be precise it's
in Garland theme for example. Try removing that and you will see that "Site map" will disappear from the page.
If he wants css add to the title he has to add a class to that in his page.tpl.php file inside his theme folder. If he needs a special case for the sitemap than he has to create page-sitemap.tpl.php and clear the cache after.
Comment #5
hass CreditAttribution: hass commentedThis IS possible with theming as you said yourself (e.g. page-sitemap.tpl.php template). I know what I'm talking about. And "Options" are NOT text.
Comment #6
tic2000 CreditAttribution: tic2000 commentedWell, if you know what you're talking about you should know that the issue you said this is a duplicate of only outputs the $content variable in page.tpl.php and in some cases not even all of $content and it has nothing to do with the title.
Providing a page-sitemap.tpl.php is not this module job, or any module job mainly because the user needs to copy that file in his theme folder anyway.
All this module can do is to provide a field for the user to set a custom title which can override the default title. This is the only reason I let this issue open and maybe someone can provide a patch for it.
This is also a feature request and should go to the last dev version.
Comment #7
tic2000 CreditAttribution: tic2000 commentedNo feature request can be solved without a patch.
Comment #8
hass CreditAttribution: hass commentedThere is a context sensitive bug that need to be fixed:
Wrong:
Correct:
Comment #9
tic2000 CreditAttribution: tic2000 commentedAnd why would that be a context sensitive bug?
Comment #10
tic2000 CreditAttribution: tic2000 commentedComment #11
hass CreditAttribution: hass commentedThe string "site map page" wasn't translatable in the context of the sentence.
Code wise RTBC.
Comment #12
hass CreditAttribution: hass commentedWorks as advertised.
Comment #13
frjo CreditAttribution: frjo commentedCommitted to 6-2-dev.
I removed the
arg(0) == 'sitemap'
since this is in the page callback.Thanks a lot for helping out with this module! It's surprisingly popular so I feel very good about getting it more polished and useful.
Comment #14
tic2000 CreditAttribution: tic2000 commentedIt's in the page callback, but some one may decide to call the function directly in his own module and maybe doesn't want the same page title.