Options for title of sitemap page

#247077: Add (node/block)-site_map.tpl.php support for custom theming

This 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.

No. 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

Stop 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

 if ($title): print '<h2'. ($tabs ? ' class="with-tabs"' : '') .'>'. $title .'</h2>'; endif; 

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.

can't have any css applied to it

This 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.

Well, 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.

No feature request can be solved without a patch.

There is a context sensitive bug that need to be fixed:

Wrong:

t('Page title that will be used on the !link', array('!link' => l('site map page', 'sitemap')))

Correct:

t('Page title that will be used on the <a href="!link">site map page</a>.', array('!link' => url('sitemap')))

And why would that be a context sensitive bug?

The string "site map page" wasn't translatable in the context of the sentence.

Code wise RTBC.

Works as advertised.

Committed 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.

It'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.