A minor bug I'm sure, but just tought I'd point it out :
In the bottom left corner where there is a "Drigg Powered" logo, on certain pages it does not show up and instead just shows a text link which says "Drigg".
Here it is reproduced on drigg.org :
Page with the logo : http://www.drigg.org/
Page where logo disappears : http://www.drigg.org/published/top24h
Comments
Comment #1
alliax commentedI'm guessing that's because the img src is relative :
< img src="sites/www.drigg.org/themes/drigg_theme/img/drigg_80_15.gif" alt="Drigg" >
it should rather be absolute with the additional / at the start like this :
< img src="/sites/www.drigg.org/themes/drigg_theme/img/drigg_80_15.gif" alt="Drigg" >
Comment #2
mercmobily commentedHi,
It actually needs the Drupal function that returns the theme's directory. I am away at the moment... can you please look it up for me and send me some working code?
That would be awesome... I am taken 1000% by FriendList/OPENsocial/Etc. at the moment. Any extra help with Drigg would be awesome (although Drigg is in a code freeze that is lasting a little too long as well... :-( )
Merc.
Comment #3
philbar commentedRelated Topic in the Drupal support forum:
http://drupal.org/node/52520
Comment #4
philbar commentedChange the following...
<a href="http://www.drigg-code.org"><img src="<?php print path_to_theme()?>/img/drigg_80_15.gif" alt="Drigg" /></a>...to this:
<a href="http://www.drigg-code.org"><img src="<?php print $base_path . $directory .'/img/drigg_80_15.gif'; ?>" alt="Drigg" /></a>in the footer div of page.tpl.php.
Comment #5
philbar commentedThis is a simple fix merc.
Comment #6
mercmobily commentedHi,
Phil, I don't get why path_to_theme() is not working... what does it contain, and what is it *supposed* to contain in a rendered page?
Are we just missing a leading "/" before path_to_theme()...?
Merc.
Comment #7
philbar commentedCommit #221370 by philbar at 19:36
Comment #8
philbar commentedMerc,
I committed it to the 6.x branch. Shouldn't we also fix bugs in 5.x? Or would you prefer to mark that branch as "Unsupported"?
Comment #9
philbar commentedComment #10
mercmobily commentedHi,
THANK YOU!
Hey, please make 10000% sure that things are 1000000% fixed when you commit. I am sure you did... but just in case :D
Philbar, with D7 coming up, I really think we should make it clear that we are concentrating on D6 now -- and let's get it done ASAP so that we can forget about the D5 version of the module. Few people are still using D5 now...
Bye,
Merc.
Comment #11
mercmobily commented