Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

More fedd back Please ....


whitehawk43

Recommended Posts

You're not getting off that easy Ben! :lol:

 

I didn't use a fine tooth comb, rather a quick brush thru LOL.

 

Lose the category counts in the menu, they slow things down and aren't really needed.

Lose those ugly "->" in the category menu.

Make sure the category links do not break to another line, looks terrible when they do.

Examples: "Bathroom Decor & Unique Accessor" why not just "Bathroom Decor" also "Purchase Our Catalog Fall 2008" maybe something like: "Purchase our Catalog".

Contact Us page is broken or rather bent out of shape.

Put a little space between the logo's in your footer, easily done by adding   a few times.

 

Just a "personal pet peeve of mine" that I see on a lot of sites. The header navigation to the right: My Account | Cart Contents | Checkout. If I'm a new customer why do I see a link to my account? Take a looky at mine (Link in my profile page)(site still under construction).

 

Nice work so far. ;)

 

Well Well Well ..........

 

That much you think sure trying to keep me from taking a break ......... :lol:

 

ok contact page fixed easy ;)

as far as the Make sure the category links do not break to another line.......

working on it the titles used are for SEO info placement so on going work in progress........

 

and let me konw how to remove the header navigation to the right off the main page like your setup ...........;)

 

Thanks

Ben

Link to comment
Share on other sites

  • 2 weeks later...
You're not getting off that easy Ben! :lol:

 

I didn't use a fine tooth comb, rather a quick brush thru LOL.

 

Lose the category counts in the menu, they slow things down and aren't really needed.

Lose those ugly "->" in the category menu.

Make sure the category links do not break to another line, looks terrible when they do.

Examples: "Bathroom Decor & Unique Accessor" why not just "Bathroom Decor" also "Purchase Our Catalog Fall 2008" maybe something like: "Purchase our Catalog".

Contact Us page is broken or rather bent out of shape.

Put a little space between the logo's in your footer, easily done by adding   a few times.

 

Just a "personal pet peeve of mine" that I see on a lot of sites. The header navigation to the right: My Account | Cart Contents | Checkout. If I'm a new customer why do I see a link to my account? Take a looky at mine (Link in my profile page)(site still under construction).

 

Nice work so far. ;)

jhande

 

can you help me with where to make the changes you said

 

Thanks

Ben

Link to comment
Share on other sites

Guest! Need to create an account? to order from our catalog

of Unique Wholesale Home Decor and Accessories....

Returning Costumers Log In Please ? to place a Order...

 

I would change these:

 

Instead of Guest! write "Guests"

Take the "!" out

 

create an account?

Take the "?" out

 

Log In Please?

Write it lower case and take the "?" out

 

"place a order"

place an order

 

These are small things, but very important to look professional!

You can't allow yourself grammar errors!

Link to comment
Share on other sites

Sorry for my absence Ben, been busy making a mess of my site. :blush:

 

Lose the category counts in the menu, they slow things down and aren't really needed.

Admin Panel > Configuration > My Store - Show Category Counts = false.

 

Lose those ugly "->" in the category menu.

catalog\includes\boxes - categories.php

Look for (starting around) line 41:

	if (tep_has_category_subcategories($counter)) {
  $categories_string .= '->';

Change the -> to something else or possibly comment the line out. I changed mine to a space with a plus sign = $categories_string .= ' +';

After you make a change, go to your Admin Panel > Tools > Cache Control > Categories Box and click on the Action triangle. That will clear the cache so when you refresh your page you will see the change.

 

Make sure the category links do not break to another line, looks terrible when they do.

I see you did some rewording. Don't forget you can also change the width of the columns when needed.

File catalog\includes\application_top.php around line 58:

// customization for the design layout
 define('BOX_WIDTH', 215); // how wide the boxes should be in pixels (default: 125)

As you can see, mine is set to a width of 215 pixels. Change it to your liking and needs.

 

Put a little space between the logo's in your footer, easily done by adding   a few times.

I see you added the space, good for you. Just a suggestion if you want to line things up a bit. Take the code you installed for the icons and move it to your footer. You could line it up something like this:

<!-- footer //-->


<table border="0" width="100%" cellspacing="0" cellpadding="0">
 <tr>
<td align="center" valign="middle"><!-- GeoTrust QuickSSL [tm] Smart Icon tag. Do not edit. -->
<script LANGUAGE="JavaScript" TYPE="text/javascript" SRC="//smarticon.geotrust.com/si.js"></SCRIPT>
<!-- end GeoTrust Smart Icon tag --></td>
<td align="center" valign="middle" class="smallText">Copyright © 2008 <a href="<A href="http://mosgifts.com/index.php?osCsid=d3d5a863e780161098d65fa54c3cf4d4">Mo's">http://mosgifts.com/index.php?osCsid=d3d5a863e780161098d65fa54c3cf4d4">Mo's</A> Gifts & Things</a>
Powered by <a href="<A href="http://www.oscommerce.com">http://www.oscommerce.com</A>" target="_blank">osCommerce</a></td>
<td align="center" valign="middle"><img src="<A href="https://mosgifts.com/images/credit_card.gif">https://mosgifts.com/images/credit_card.gif</A>"></td>
 </tr>
</table>
<!-- footer_eof //-->

 

Just a "personal pet peeve of mine" that I see on a lot of sites. The header navigation to the right: My Account | Cart Contents | Checkout. If I'm a new customer why do I see a link to my account? Take a looky at mine (Link in my profile page)(site still under construction).

 

File catalog\includes\header.php around line 78 I have this code:

<td height="20" align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>

Sorry I don't have a copy of the original, but you should be able to find which code to replace.

 

Two more little suggestions. :blush:

 

The word Guest! on the home page gets lost. Change to a darker color to make it easier to see.

The View All Products link in the categories, change the allproductsBox class to match your boxText and link classes. Better yet, simply drop the allproductsBox class and let it share the class for the rest of the category box.

 

Site is looking good ;)

 

Hope that helps some and again, sorry for the late reply.

- :: Jim :: -

- My Toolbox ~ Adobe Web Bundle, XAMPP & WinMerge | Install ~ osC v2.3.3.4 -

Link to comment
Share on other sites

Jim

 

having trouble making the changes below things don't work out here is my header php file

 

( <?php

/*

$Id: header.php 1739 2007-12-20 00:52:16Z hpdl $

 

osCommerce, Open Source E-Commerce Solutions

http://www.oscommerce.com

 

Copyright © 2003 osCommerce

 

Released under the GNU General Public License

*/

 

// check if the 'install' directory exists, and warn of its existence

if (WARN_INSTALL_EXISTENCE == 'true') {

if (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/install')) {

$messageStack->add('header', WARNING_INSTALL_DIRECTORY_EXISTS, 'warning');

}

}

 

// check if the configure.php file is writeable

if (WARN_CONFIG_WRITEABLE == 'true') {

if ( (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) && (is_writeable(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) ) {

$messageStack->add('header', WARNING_CONFIG_FILE_WRITEABLE, 'warning');

}

}

 

// check if the session folder is writeable

if (WARN_SESSION_DIRECTORY_NOT_WRITEABLE == 'true') {

if (STORE_SESSIONS == '') {

if (!is_dir(tep_session_save_path())) {

$messageStack->add('header', WARNING_SESSION_DIRECTORY_NON_EXISTENT, 'warning');

} elseif (!is_writeable(tep_session_save_path())) {

$messageStack->add('header', WARNING_SESSION_DIRECTORY_NOT_WRITEABLE, 'warning');

}

}

}

 

// check session.auto_start is disabled

if ( (function_exists('ini_get')) && (WARN_SESSION_AUTO_START == 'true') ) {

if (ini_get('session.auto_start') == '1') {

$messageStack->add('header', WARNING_SESSION_AUTO_START, 'warning');

}

}

 

if ( (WARN_DOWNLOAD_DIRECTORY_NOT_READABLE == 'true') && (DOWNLOAD_ENABLED == 'true') ) {

if (!is_dir(DIR_FS_DOWNLOAD)) {

$messageStack->add('header', WARNING_DOWNLOAD_DIRECTORY_NON_EXISTENT, 'warning');

}

}

 

if ($messageStack->size('header') > 0) {

echo $messageStack->output('header');

}

?>

<table width="875" cellspacing="0" cellpadding="0" align="center">

<tr>

<td class="outline">

<?php /*** Begin Header Tags SEO ***/ ?>

<td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'mos1.jpg', (tep_not_null($header_tags_array['logo_text']) ? $header_tags_array['logo_text'] : STORE_NAME)) . '</a>'; ?></td>

<?php /*** End Header Tags SEO ***/ ?>

 

</td>

</table>

<table border="0" width="875" align="center" cellspacing="0" cellpadding="0">

<tr class="headerNavigation">

<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_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>

</table>

<?php

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

?>

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

<tr class="headerError">

<td class="headerError"><?php echo htmlspecialchars(stripslashes(urldecode($HTTP_GET_VARS['error_message']))); ?></td>

</tr>

</table>

<table

<?php

}

 

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

?>

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

<tr class="headerInfo">

<td class="headerInfo"><?php echo htmlspecialchars(stripslashes(urldecode($HTTP_GET_VARS['info_message']))); ?></td>

</tr>

</table>

</td>

</tr>

</table>

<?php

}

?>

)

 

sorry just can't make the change below work

 

File catalog\includes\header.php around line 78 I have this code:

<td height="20" align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>

Sorry I don't have a copy of the original, but you should be able to find which code to replace.

Link to comment
Share on other sites

The View All Products link in the categories, change the allproductsBox class to match your boxText and link classes. Better yet, simply drop the allproductsBox class and let it share the class for the rest of the category box.

 

Jim I am not sure what you mean here

 

Ben

Link to comment
Share on other sites

Jim I am not sure what you mean here

 

Ben

 

Hey Ben,

 

I'll start with trying to explain the easier one first. Still on my first cup of coffee this morning LOL. -_-

 

In your Wholesale Catalog infobox (categories menu) all the links are white except View All Products which is black. It's not bad but personally I would want to change it to white to match the other links, but that's up to you.

 

To do so, your stylesheet.css controls the look and behavior, there are a few classes defining them.

But when you look at the code where you added the View All Products link, there is a class allproductsBox called out which controls the look of that link. One, you don't need to have the allproductsBox class added, or change that class to match.

 

Easiest fix: open stylesheet.css, find a.allproductsBox and change #000000 to #FFFFFF.

 

Sorry if I rambled needlessly. I got lost in my thoughts. Need another cup of coffee. :blush:

 

Back in a minute to look at the other problem.

- :: Jim :: -

- My Toolbox ~ Adobe Web Bundle, XAMPP & WinMerge | Install ~ osC v2.3.3.4 -

Link to comment
Share on other sites

Jim

 

having trouble making the changes below things don't work out here is my header php file

 

( <?php

/*

$Id: header.php 1739 2007-12-20 00:52:16Z hpdl $

 

osCommerce, Open Source E-Commerce Solutions

http://www.oscommerce.com

 

Copyright © 2003 osCommerce

 

Released under the GNU General Public License

*/

 

// check if the 'install' directory exists, and warn of its existence

if (WARN_INSTALL_EXISTENCE == 'true') {

if (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/install')) {

$messageStack->add('header', WARNING_INSTALL_DIRECTORY_EXISTS, 'warning');

}

}

 

// check if the configure.php file is writeable

if (WARN_CONFIG_WRITEABLE == 'true') {

if ( (file_exists(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) && (is_writeable(dirname($HTTP_SERVER_VARS['SCRIPT_FILENAME']) . '/includes/configure.php')) ) {

$messageStack->add('header', WARNING_CONFIG_FILE_WRITEABLE, 'warning');

}

}

 

// check if the session folder is writeable

if (WARN_SESSION_DIRECTORY_NOT_WRITEABLE == 'true') {

if (STORE_SESSIONS == '') {

if (!is_dir(tep_session_save_path())) {

$messageStack->add('header', WARNING_SESSION_DIRECTORY_NON_EXISTENT, 'warning');

} elseif (!is_writeable(tep_session_save_path())) {

$messageStack->add('header', WARNING_SESSION_DIRECTORY_NOT_WRITEABLE, 'warning');

}

}

}

 

// check session.auto_start is disabled

if ( (function_exists('ini_get')) && (WARN_SESSION_AUTO_START == 'true') ) {

if (ini_get('session.auto_start') == '1') {

$messageStack->add('header', WARNING_SESSION_AUTO_START, 'warning');

}

}

 

if ( (WARN_DOWNLOAD_DIRECTORY_NOT_READABLE == 'true') && (DOWNLOAD_ENABLED == 'true') ) {

if (!is_dir(DIR_FS_DOWNLOAD)) {

$messageStack->add('header', WARNING_DOWNLOAD_DIRECTORY_NON_EXISTENT, 'warning');

}

}

 

if ($messageStack->size('header') > 0) {

echo $messageStack->output('header');

}

?>

<table width="875" cellspacing="0" cellpadding="0" align="center">

<tr>

<td class="outline">

<?php /*** Begin Header Tags SEO ***/ ?>

<td valign="middle"><?php echo '<a href="' . tep_href_link(FILENAME_DEFAULT) . '">' . tep_image(DIR_WS_IMAGES . 'mos1.jpg', (tep_not_null($header_tags_array['logo_text']) ? $header_tags_array['logo_text'] : STORE_NAME)) . '</a>'; ?></td>

<?php /*** End Header Tags SEO ***/ ?>

 

</td>

</table>

<table border="0" width="875" align="center" cellspacing="0" cellpadding="0">

<tr class="headerNavigation">

<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_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>

</table>

<?php

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

?>

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

<tr class="headerError">

<td class="headerError"><?php echo htmlspecialchars(stripslashes(urldecode($HTTP_GET_VARS['error_message']))); ?></td>

</tr>

</table>

<table

<?php

}

 

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

?>

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

<tr class="headerInfo">

<td class="headerInfo"><?php echo htmlspecialchars(stripslashes(urldecode($HTTP_GET_VARS['info_message']))); ?></td>

</tr>

</table>

</td>

</tr>

</table>

<?php

}

?>

)

 

sorry just can't make the change below work

 

File catalog\includes\header.php around line 78 I have this code:

<td height="20" align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>

Sorry I don't have a copy of the original, but you should be able to find which code to replace.

 

Ok, I didn't need that second cup of coffee after all. :lol:

 

Find this (which is highlighted in blue above):

<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_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>

 

And replace with this:

<td align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>

 

;)

- :: Jim :: -

- My Toolbox ~ Adobe Web Bundle, XAMPP & WinMerge | Install ~ osC v2.3.3.4 -

Link to comment
Share on other sites

WOW Ben, that's not right at all... :o

 

Ok, when I view your header.php code in my browser I see this:

 

<table width="875" cellspacing="0" cellpadding="0" align="center">
<tr>
<td class="outline">
<td valign="middle"><a href="<A href='http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><img'>http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><img</A> src="images/mos1.jpg" border="0" alt="Unique Wholesale Home Decor and Accessories" title=" Unique Wholesale Home Decor and Accessories " width="875" height="203"></a></td>
</td>
</table>
<table border="0" width="875" align="center" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <a href="<A href="http://mosgifts.com">http://mosgifts.com</A>" class="headerNavigation"></a> » <a href="<A href="http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36">http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36</A>" class="headerNavigation">Home</a></td>
 <td align="right" class="headerNavigation"><font size="2">Welcome To our Store...
Hope you enjoy  our full line of
Wholesale Home Decor and Unique Accessories...
</P> <P><span class="greetUser"><font color="black">Guest</font color></span> Need to <a href="<A href='https://mosgifts.com/login.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>create'>https://mosgifts.com/login.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>create</A> an account</u></a> to order from our catalog of Unique Wholesale Home Decor and Accessories....
Returning Costumers <a href="<A href='https://mosgifts.com/create_account.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>log'>https://mosgifts.com/create_account.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>log</A> in please </u></a> to place an Order...   </td>
</table>

 

It looks like you copied a bunch of code from your index.php and there is no sign of the code I posted for you to use.

 

Go back to your header.php file and look for the above code (you won't see any url addresses) starting with that first <table> tag as shown above and ending with the last </table> tag as shown above.

 

Replace it with this code:

 

<table boder="0"  width="875" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <?php echo $breadcrumb->trail(' » '); ?></td>
<td align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>
 </tr>
</table>

 

That should do it.

- :: Jim :: -

- My Toolbox ~ Adobe Web Bundle, XAMPP & WinMerge | Install ~ osC v2.3.3.4 -

Link to comment
Share on other sites

WOW Ben, that's not right at all... :o

 

Ok, when I view your header.php code in my browser I see this:

 

<table width="875" cellspacing="0" cellpadding="0" align="center">
<tr>
<td class="outline">
<td valign="middle"><a href="<A href='http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><img'>http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><img</A> src="images/mos1.jpg" border="0" alt="Unique Wholesale Home Decor and Accessories" title=" Unique Wholesale Home Decor and Accessories " width="875" height="203"></a></td>
</td>
</table>
<table border="0" width="875" align="center" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <a href="<A href="http://mosgifts.com">http://mosgifts.com</A>" class="headerNavigation"></a> » <a href="<A href="http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36">http://mosgifts.com/index.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36</A>" class="headerNavigation">Home</a></td>
 <td align="right" class="headerNavigation"><font size="2">Welcome To our Store...
Hope you enjoy  our full line of
Wholesale Home Decor and Unique Accessories...
</P> <P><span class="greetUser"><font color="black">Guest</font color></span> Need to <a href="<A href='https://mosgifts.com/login.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>create'>https://mosgifts.com/login.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>create</A> an account</u></a> to order from our catalog of Unique Wholesale Home Decor and Accessories....
Returning Costumers <a href="<A href='https://mosgifts.com/create_account.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>log'>https://mosgifts.com/create_account.php?osCsid=de2d07cbe30bb1975b6e36c77bd1ea36"><u>log</A> in please </u></a> to place an Order...   </td>
</table>

 

It looks like you copied a bunch of code from your index.php and there is no sign of the code I posted for you to use.

 

Go back to your header.php file and look for the above code (you won't see any url addresses) starting with that first <table> tag as shown above and ending with the last </table> tag as shown above.

 

Replace it with this code:

 

<table boder="0"  width="875" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <?php echo $breadcrumb->trail(' » '); ?></td>
<td align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>
 </tr>
</table>

 

That should do it.

 

 

hey jim

lol I knew that was not what we were shooting for but all I did was remove one line of code that you had highlighted in blue and replaced it with the one you siad I should do

 

so to your ? looking like I copyed index.php code nope lol

 

will give it a try and see what happens

 

Ben

Link to comment
Share on other sites

Hey jim

 

no luck that did not make any changes

just moved it from the right to kinda the center

 

so I am not sure what is going on

 

I put the code back the way it was before I made the changes

 

Ben

 

Hey Ben,

 

Gee... I don't understand why it doesn't work, it's simply replacing one set of links for another. :huh:

 

Your code:

<table border="0" width="875" align="center" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <a href="<A href="http://mosgifts.com">http://mosgifts.com</A>" class="headerNavigation"></a> » <a href="<A href="http://mosgifts.com/index.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6">http://mosgifts.com/index.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6</A>" class="headerNavigation">Home</a></td>
<td align="right" class="headerNavigation"><a href="<A href="https://mosgifts.com/account.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6">https://mosgifts.com/account.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6</A>" class="headerNavigation">My Account</a>  |  <a href="<A href="http://mosgifts.com/shopping_cart.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6">http://mosgifts.com/shopping_cart.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6</A>" class="headerNavigation">Cart Contents</a>  |  <a href="<A href="https://mosgifts.com/checkout_shipping.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6">https://mosgifts.com/checkout_shipping.php?osCsid=bbc1b34d4fd8cd74f6cbb4d2b023f8a6</A>" class="headerNavigation">Checkout</a>   </td>
</table>

 

New code:

<table boder="0"  width="875" cellspacing="0" cellpadding="0">
 <tr class="headerNavigation">
<td class="headerNavigation">  <?php echo $breadcrumb->trail(' » '); ?></td>
<td align="right" class="headerNavigation"><?php echo tep_customer_greeting(); ?>    <?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>  |  <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_CHECKOUT_SHIPPING, '', 'SSL'); ?>" class="headerNavigation"><?php echo HEADER_TITLE_CHECKOUT; ?></a>   <?php } ?></td>
 </tr>
</table>

 

It's only the second table cell (<td>) that is changing.

 

Oh by the way, you are missing a </tr> before the </table> tag in your code.

 

Sorry it didn't work for you, I don't understand why not.

- :: Jim :: -

- My Toolbox ~ Adobe Web Bundle, XAMPP & WinMerge | Install ~ osC v2.3.3.4 -

Link to comment
Share on other sites

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...