When a banner has width size of 728px, the user/uid/affiliate banner display will be broken in most themes because the banner + the text area for the banner code is displayed side by side.
I changed the code for displaying the banner to ( function _uc_affiliate2_display_banners()):
$line[] = $banner . ''. $banner_link .'';
and the header of the table to :
$header[] = array('field' => 'banner_preview', 'data' => t('Banner Preview & Banner Code'));
But I think the proper way is to implement theme hook since this matter is connected to a theme?
Is it ok if I create a hook_theme for this function?
Comments
Comment #1
bojanz commentedSure, do it and post it here.
Comment #2
duckzland commentedSorry that I cant provide a proper patch, No linux available.
anyway,
I think that is sufficient to make the function themeable
Comment #4
jazzdrive3 commentedYeah, I'm getting this problem too. The code to copy is pushed way off the side of the page. Broken.
Is something like this going to be committed soon?
Thanks.
Comment #5
bojanz commentedIf someone provides a proper and tested patch, yes.
Otherwise, no, since development pretty much stopped.
Comment #6
tmwagner commentedOne solution that I think would really help, would be the ability to CHOOSE which image is displayed as a thumbnail.
The Affiliate Banner Template "suggests" that there is some flexibility (i.e., we can insert as many (tags?) as we wish...)
The problem is, the banner uses the default main image. Most instances would tend to use an image as large as practicable... that, in turn breaks the affiliate thumbnail concept.
Comment #7
avpadernoI am deleting the content of unrelated files added by a spammer. I am also closing this issue, since Drupal 6 isn't supported anymore.
I apologize for bumping this issue.