The block title is currently hardcoded :( as a link. In the way I use the module I do not want a link to the node at all. The attached patch adds a theme function to allow override.

CommentFileSizeAuthor
nodeasblock.theme_title.patch1.22 KBsutharsan

Comments

Tobias Weibel’s picture

Tobias Weibel commented

Hi Sutharsan

Thanks for your patch. I had the same issue. It's solved now.

@maintainers: Could this patch be included in the standard module?

Thanks
Tobias

jennycita’s picture

jennycita commented

+1

aschiwi’s picture

aschiwi commented

Hey Sutharsan, I wanted to RTBC your patch but could you review it? I'm trying to patch the current dev but it doesn't really do anything on a brand new drupal. I'm guessing things have changed in the code. If it's not too hard it would probably be best to make it a setting: Do you want the block title to be linked to the node? I would be fine without the setting since I only want an unlinked title. If it's not a setting it could also be done in the template file, which is currently not possible. I'll be happy to test a revised patch as soon as it's there.

sutharsan’s picture

sutharsan commented

From a user's point of view is does not change anything indeed. The patch provides the possibility for theme override of the block title. I am not a great fan of loads of features. Better choose sensible defaults and give an option to override in the theming layer (in code). But even if an option would be added it is the Drupal way to provide theme override options.

jerdavis’s picture

jerdavis
Location Minneapolis, MN
commented
Status: Needs review » Reviewed & tested by the community

This change looks good to me, I agree with you Sutharsan about not requiring a UI configuration for every single thing. Linking the title is a sensible default, but being able to undo that easily in the theme layer is a good option.