The task here is to allow icons to be overridden in the theme, and that means the theme functions in OTW need to be rewritten.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jenlampton’s picture

Assigned: jenlampton » Unassigned
Status: Active » Needs review
FileSize
4.05 KB

Okay, here's a rewrite of the theme functions that make the icons easier to override.

jenlampton’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

greenSkin’s picture

Status: Closed (fixed) » Needs review
FileSize
2.03 KB

I like that you've added the theme functions, definitely makes things easier to alter. I'd like to offer one more patch against the 7.x-1.x version that simplifies using custom images without having to write any code. I've added a textfield to the settings fieldset in the block's configuration that allows setting the images folder path. Leaving it blank will default to the images folder within the module.

jenlampton’s picture

Status: Needs review » Closed (fixed)

I think that's a great idea, can you open a new issue and attach the patch there (since this is a different request)? Thanks!