[drupal-devel] [bug] bluemarine mistakenly loads home page every time

[chx]

Issue status update for 
http://drupal.org/node/29283
Post a follow up: 
http://drupal.org/project/comments/add/29283

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     bug reports
 Priority:     normal
 Assigned to:  moshe weitzman
 Reported by:  moshe weitzman
 Updated by:   chx
-Status:       patch (code needs review)
+Status:       patch (ready to be committed)
 Attachment:   http://drupal.org/files/issues/logo_2.patch (1.48 KB)

The patch was OK but was not from Drupal directory. I rerolled and I
think it's a go.




chx



Previous comments:
------------------------------------------------------------------------

Sun, 21 Aug 2005 01:49:53 +0000 : moshe weitzman

Attachment: http://drupal.org/files/issues/logo.patch (830 bytes)

if admin chooses no logo in theme config, bluemarine still lays down an
img tag which points back to the site home page. thats a sure way to
increase your load and bandwidth. yikes!




------------------------------------------------------------------------

Sun, 21 Aug 2005 01:53:19 +0000 : moshe weitzman

pushbutton already avoids this mistake so i copied its code.




------------------------------------------------------------------------

Sun, 21 Aug 2005 11:51:27 +0000 : Uwe Hermann

Attachment: http://drupal.org/files/issues/logo_0.patch (791 bytes)

The patch is broken. Here's a fixed one.




------------------------------------------------------------------------

Sun, 21 Aug 2005 12:01:01 +0000 : fajerstarter

When your at it: shouldn't .., .. etc. be wrapped with  to remove the
empty  when not in use. Cleaner XHTML output. Or is it too expesive to
check that for each div?




------------------------------------------------------------------------

Sun, 21 Aug 2005 12:06:46 +0000 : fajerstarter

My mistake :) I meant:


Shouldn't <div class='site-slogan'>... </div> , <div id="secondary">...
</div> etc. be wrapped with  to remove the empty when not in use.
Cleaner XHTML output. Or is it too expesive to check that for each div?




------------------------------------------------------------------------

Sun, 21 Aug 2005 12:08:33 +0000 : fajerstarter

... wrapped with <?php if ... ?> ...


sorry...  BTW, the preview function doesn't work :)




------------------------------------------------------------------------

Sun, 21 Aug 2005 13:07:19 +0000 : Uwe Hermann

Attachment: http://drupal.org/files/issues/logo_1.patch (1.43 KB)

Sure, why not. I don't think performance will suffer too much. Here's an
updated patch.