Hi,

Nice looking theme, thank you. I've been trying to add a custom logo in configuration for this theme, but unfortunately it is not storing. I have around ten other themes I tried on the same site, and they all work fine when adding custom logos. Any help would be appreciated.

Thanks,
Joseph

Comments

josephgut’s picture

Title: Unable to Add Custom Icon » Unable to Add Custom Logo
kiwi_steve’s picture

confirmed this myself... I will hack the theme to get it working since I like it, but it would be nice if it worked. If I figure it out (which I probably won't, I'm not so flash with PHP) I'll post back a fix.

Nice theme tho.

Steve

citronica’s picture

Yup, me too.

kristin.gasser’s picture

Having the same trouble. Even if I replace the A Vitamin logo with my logo, forced to the same dimensions & same file name, it doesn't go away. If I turn off the logo in the theme configuration, the logo image still persists. I've been able to hide it (display:none;), place a background image behind it (also in the css) but not replace it in any way.

Love the theme - would really like to use it. Many thanks!

kristin.gasser’s picture

As a temporary work-around, I backed up the logo.png file in the avitamin folder and replaced it with my logo (named logo.png to replace the file completely). I also changed the logo div dimensions to fit my image. So far, so good.

foupfeiffer’s picture

Component: Code » User interface
Priority: Normal » Major
Issue tags: +patch, +Logo, +page.tpl.php, +vitamin theme

I also have had issues with the A Vitamin theme logo.

Very nice looking theme and sub menus are easy but it's weird that the checkbox doesn't seem to control the logo whatsoever.

The "hack fix" of replacing
/sites/all/themes/avitamin-6.x-1.4/avitamin/logo.png
with my own custom image isn't what I was looking for, especially as I wanted to disable the logo and instead use an image block in the header (so the menu is actually above the logo).

My first venture into the open source world, can this be applied as a patch?

page.tpl.php

<div id="logo"><!--LOGO-->
       <?php
       if ( $logo ) 
       {
           print '<a id="logo-uri" href="' . base_path() . '" title="' . $site_name . '"><img id="logo-img" src="' . base_path() . path_to_theme() . '/logo.png" alt="' . $site_name . '" title="' .  $site_name . '" /></a>';
      
      
      }
      ?>
      </div><!--END LOGO-->

Basically I've taken the Garland theme as a template and improved the code in page.tpl.php in the Vitamin theme to actually use the $logo boolean. I hate the single quote next to double quote thing too but it's the proper way to do a mixed print statement of variables and constants in php/html.

It works fine on my installation (checkbox in /admin/build/themes/settings/avitamin now controls whether the logo appears or not).

Dunno about the rest (mission statement et. al.) but page.tpl will need more work to bring it inline with Garland/Drupal's easy to configure via admin pages goals.

FiNeX’s picture

Status: Active » Fixed

Hi! I'm sorry for answering after some weeks but I've been busy. I've released a new version (6.x-1.5) with the fix.

FiNeX’s picture

I'm sorry for the late answer. I've just released AVitamin 6.x-1.5 with the support for the custom logo :-)

shp’s picture

Version: 6.x-1.4 » 6.x-1.5
Status: Fixed » Active

Hmm, custom logo seems not to be working on 6.x-1.5. In page.tpl.php the logo's path is "<? php print base_path() . path_to_theme(); ?>/logo.png"

wheresjbob’s picture

Any update on this? I also like the theme. Thanks for creating it.

IrishGringo’s picture

I cannot get the logo to work either. I try to upload a logo... no joy
I think part of the problem is that the URL to the logo is:
/sites/all/themes/avitamin/logo.png

THE PROBLEM IS THIS CODE in page.tpl.php line 21.

print base_path(); " title=" print $site_name ">Only local images are allowed. print base_path() . path_to_theme(); /logo.png" alt=" print $site_name " title=" print $site_name " />

-->

THE SOLUTION FOR ME IS: replace that code with this code:

It worked for me...

ruifigueiredo’s picture

Hi,
remove the div id = "logo" and replace with:

     <div id="logo"><!--LOGO-->
      <?php if ($logo) { ?>
        <a id="logo-uri" href="<?php print base_path(); ?>" title="<?php print $site_name ?>"><img id="logo-img" src="<?php print $logo; ?>" alt="<?php print $site_name ?>" title="<?php print $site_name ?>" /></a>
      <?php } else { ?>
        <a id="logo-uri" href="<?php print base_path(); ?>" title="<?php print $site_name ?>"><img id="logo-img" src="<?php print base_path() . path_to_theme(); ?>/logo.png" alt="<?php print $site_name ?>" title="<?php print $site_name ?>" /></a>
      <?php } ?>
      </div><!--END LOGO-->

regards;)
Rui Figueiredo

FiNeX’s picture

Status: Active » Fixed

I'm sorry, but when I've uploaded the 1.5 version I've done a mess with cvs... I've re-tagged and repackaged it.

Status: Fixed » Closed (fixed)
Issue tags: -patch, -Logo, -page.tpl.php, -vitamin theme

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