Closed (fixed)
Project:
A Vitamin
Version:
6.x-1.5
Component:
User interface
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Anonymous (not verified)
Created:
26 Jun 2010 at 04:02 UTC
Updated:
3 Jul 2024 at 19:10 UTC
Jump to comment: Most recent
Comments
Comment #1
Anonymous (not verified) commentedComment #2
kiwi_steve commentedconfirmed 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
Comment #3
citronica commentedYup, me too.
Comment #4
kristin.gasser commentedHaving 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!
Comment #5
kristin.gasser commentedAs 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.
Comment #6
foupfeiffer commentedI 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
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.
Comment #7
finex commentedHi! 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.
Comment #8
finex commentedI'm sorry for the late answer. I've just released AVitamin 6.x-1.5 with the support for the custom logo :-)
Comment #9
shp commentedHmm, 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"
Comment #10
wheresjbob commentedAny update on this? I also like the theme. Thanks for creating it.
Comment #11
irishgringo commentedI 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 base_path() . path_to_theme(); /logo.png" alt="
print $site_name">print $site_name" title="print $site_name" />-->
THE SOLUTION FOR ME IS: replace that code with this code:
if ($logo):print t('Home')">print t('Home')"/>endif;It worked for me...
Comment #12
ruifigueiredo commentedHi,
remove the div id = "logo" and replace with:
regards;)
Rui Figueiredo
Comment #13
finex commentedI'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.