White line on right side of logo

JaciD · November 11, 2009

I have a white line on the side of my logo. It may be a cellpadding issue but I looked everywhere to find the problem.

<tr>

<?php

?>

<td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'store_logo.png', STORE_NAME) . '</a>'; ?></td>

<td align="right" valign="bottom"><?php //echo '<a href="' . tep_href_link(FILENAME_ACCOUNT, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_account.gif', HEADER_TITLE_MY_ACCOUNT) . '</a> <a href="' . tep_href_link(FILENAME_SHOPPING_CART) . '">' . tep_image(DIR_WS_IMAGES . 'header_cart.gif', HEADER_TITLE_CART_CONTENTS) . '</a> <a href="' . tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_checkout.gif', HEADER_TITLE_CHECKOUT) . '</a>'; ?> </td>

</tr>

</table>

<td class="headerNavigation"> <?php echo $breadcrumb->trail(' » '); ?></td>

<td align="right" class="headerNavigation"><?php if (tep_session_is_registered('customer_id')) { ?><a href="<?php echo tep_href_link(FILENAME_LOGOFF, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_LOGOFF; ?></a> | <?php } ?><a href="<?php echo tep_href_link(FILENAME_CREATE_ACCOUNT, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CREATE_ACCOUNT; ?></a> | <a href="<?php echo tep_href_link(FILENAME_LOGIN, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_LOGIN; ?></a> | <a href="<?php echo tep_href_link(FILENAME_ACCOUNT, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_MY_ACCOUNT; ?></a> | <a href="<?php echo tep_href_link(FILENAME_SHOPPING_CART); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CART_CONTENTS; ?></a> | <a href="<?php echo tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a> </td>

</tr>

</table>

<?php

if (isset($HTTP_GET_VARS['error_message']) && tep_not_null($HTTP_GET_VARS['error_message'])) {

?>

</tr>

</table>

<?php

}

if (isset($HTTP_GET_VARS['info_message']) && tep_not_null($HTTP_GET_VARS['info_message'])) {

?>

</tr>

</table>

<?php

}

?>

Click here to view my site.

Thank you for your consideration

Xpajun · November 11, 2009

In your script somewhere - I suspect header - you have an additional cell <td valign="bottom" align="right"> </td> , just remove this

Actually it's probably this:

<td align="right" valign="bottom"><?php //echo '<a href="' . tep_href_link(FILENAME_ACCOUNT, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_account.gif', HEADER_TITLE_MY_ACCOUNT) . '</a>  <a href="' . tep_href_link(FILENAME_SHOPPING_CART) . '">' . tep_image(DIR_WS_IMAGES . 'header_cart.gif', HEADER_TITLE_CART_CONTENTS) . '</a>  <a href="' . tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_checkout.gif', HEADER_TITLE_CHECKOUT) . '</a>'; ?>  </td>

which seems to be a repeat of your header navigation class?

JaciD · November 11, 2009

Thank you for your response. I will backup, look for it and then remove the line.

Thank you for your consideration.

JaciD · November 13, 2009

PLEASE! Could I get a second opinion about how to solve this problem. I looked at the file and the one that they pointed out was the shopping cart header file, I believe (see) above). The problem I am having is main page header (white line on right side) If someone could just direct me in the right path to find this solution, it would be highly appreciated.

Thank you for your consideration.

MrPhil · November 13, 2009

In the displayed page, your logo header is

<table border="0" width="100%" cellspacing="0" cellpadding="0">

 <tr class="header">
   <td valign="middle"><a href="http://fiveecors.com/Store/index.php"><img src="images/store_logo.png" border="0" alt="Five Eco R's LLC" title=" Five Eco R's LLC " width="1000" height="200"></a></td>
   <td align="right" valign="bottom">& nbsp;& nbsp;</td>
 </tr>
</table>

That second <td> with the two blanks (<td align="right" valign="bottom"> </td>) is what's giving you the blank cell with default (white) background. You need to search your PHP code for where that's output (look for <td align="right" valign="bottom"> and/or ) and remove it. It's probably the result of an add-on or some manual editing.

In the code you attached in an earlier post, it's probably the line Xpajun listed (starting at line 12 or so). Note that in his posting, the got changed to two blanks!

The PHP comment // is killing everything up to ?>, so you end up with:

<td align="right" valign="bottom">& nbsp;& nbsp;</td>

If you made that change, did you intend to comment out that much, or just a part of the line? Use /* and */ around sections of code you want to remove. Anyway, remove or comment out everything <td through /td>. HTML comments are .

JaciD · November 13, 2009

In header.php I have the following:

?>

<table border="0" width="100%" cellspacing="0" cellpadding="0">

<tr class="header">

<td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'store_logo.png', STORE_NAME) . '</a>'; ?></td>

<td align="right" valign="bottom"><?php //echo '<a href="' . tep_href_link(FILENAME_ACCOUNT, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_account.gif', HEADER_TITLE_MY_ACCOUNT) . '</a> <a href="' . tep_href_link(FILENAME_SHOPPING_CART) . '">' . tep_image(DIR_WS_IMAGES . 'header_cart.gif', HEADER_TITLE_CART_CONTENTS) . '</a> <a href="' . tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_checkout.gif', HEADER_TITLE_CHECKOUT) . '</a>'; ?> </td>

</tr>

So, If I understand you right I have the correct part and I should do the following (highlighted) in red):

?>

<td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'store_logo.png', STORE_NAME) . '</a>'; ?></td>

<td align="right" valign="bottom"><?php //echo '<a href="' . tep_href_link(FILENAME_ACCOUNT, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_account.gif', HEADER_TITLE_MY_ACCOUNT) . '</a> <a href="' . tep_href_link(FILENAME_SHOPPING_CART) . '">' . tep_image(DIR_WS_IMAGES . 'header_cart.gif', HEADER_TITLE_CART_CONTENTS) . '</a> <a href="' . tep_href_link(FILENAME_CHECKOUT_SHIPPING, '', 'SSL') . '">' . tep_image(DIR_WS_IMAGES . 'header_checkout.gif', HEADER_TITLE_CHECKOUT) . '</a>'; ?>/* */></td

</tr>

Just trying to verify before making changes.

Thank you for your consideration.

JaciD · November 13, 2009

OK, Thank you very much fixed that. :D

Now I have another question, please!

I was doing a add-on in regards to random products. The directions stated to:

2) Edit the text defined for variable RND_HEAD_TEXT which would be for the box title that

normally read "ProductS for %s" with %s being the current month. Find this and change as needed for your language:

case 'english':

define('RND_HEAD_TEXT', 'Engelsk tekst'); Insert your text here instead of the 'Engelsk tekst'. This means

'English text' in Danish I think.

So, I did this: (New Products for Each Month)

****random_pro.php file*****

case 'english':

define('RND_HEAD_TEXT', 'New Products for %s');

It shows the same way in the includes/langages/english/index file:

define('TABLE_HEADING_NEW_PRODUCTS', 'New Products For %s');

It looks like this: New Products for %s

Is there something else I should have done?

Thank you for your consideration.

MrPhil · November 13, 2009

The "%s" means that there should be a formatted print somewhere (printf or sprintf) using RND_HEAD_TEXT as its template, and the month name as its data (to replace "%s"). It looks like there is no such code, and is just using the RND_HEAD_TEXT string "as-is". The original code define('RND_HEAD_TEXT', 'Engelsk tekst'); does not define a field "%s" to be substituted, so maybe that wasn't the intent. If so, I don't know why they would instruct you to put "%s" in the string. It would be odd to have you hardcode the month into the string, because it would have to be updated monthly.

Look through all files touched by this add-on and see how RND_HEAD_TEXT is used. Is there some place where it's used in a printf or sprintf function call, along with the month name? If so, maybe you can tell why that new string (New Products For November) isn't being used. Which add-on is this?

JaciD · November 13, 2009

The "%s" means that there should be a formatted print somewhere (printf or sprintf) using RND_HEAD_TEXT as its template, and the month name as its data (to replace "%s"). It looks like there is no such code, and is just using the RND_HEAD_TEXT string "as-is". The original code define('RND_HEAD_TEXT', 'Engelsk tekst'); does not define a field "%s" to be substituted, so maybe that wasn't the intent. If so, I don't know why they would instruct you to put "%s" in the string. It would be odd to have you hardcode the month into the string, because it would have to be updated monthly.

Look through all files touched by this add-on and see how RND_HEAD_TEXT is used. Is there some place where it's used in a printf or sprintf function call, along with the month name? If so, maybe you can tell why that new string (New Products For November) isn't being used. Which add-on is this?

Thanks again for your help! I have decided to change the wording to "Marketplace Featured Items for this Month" instead of trying to have it automatically changing each month.

White line on right side of logo

Recommended Posts

JaciD

Xpajun

JaciD

JaciD

MrPhil

JaciD

JaciD

MrPhil

JaciD

Archived

Browse

Activity