Download & Extend
Clickpath » Issues

make the module themeable

Posted by shua.adi on March 15, 2009 at 6:03pm
Project:Clickpath
Version:6.x-1.1
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work

Issue Summary

made the module themeable with hook_theme and overrideable function which gets the block content as an array.

AttachmentSize
patch_theme.txt2.09 KB
Login or register to post comments

Comments

#1

Posted by shua.adi on March 15, 2009 at 6:07pm
Status:active» needs review

#2

Posted by coltrane on March 16, 2009 at 2:54pm
Status:needs review» needs work

Patch contains the "consider list" code from #402682: "consider" list please reroll and I'll review, I'm in favor of this feature.

#3

Posted by shua.adi on March 17, 2009 at 11:05am
AttachmentSize
patch_theme.txt 1.95 KB

#4

Posted by coltrane on March 17, 2009 at 3:12pm
Status:needs work» needs review

#5

Posted by coltrane on March 21, 2009 at 7:01pm
Status:needs review» needs work

#3 patch contains a variable_get('clickpath_block_list_as_path'). This variable isn't set in the patch and doesn't exist in the module. Was this patch rolled from a more modified version of clickpath?

#6

Posted by shua.adi on March 24, 2009 at 2:13pm

Hi,
yes sorry I missed it. i split the original into two issues. the variable was taken from the form.
I'll send it soon with the form changing inside.

Adi.