Custom Branding

I have my own branded header, logo...etc It is made up of an image and two animated gifs

I was wondering if editing the branding section of the tpl_header.php file would work

<!--bof-branding display-->
<div id="logoWrapper">
<PHP? include '..../custom_brand.php' ?>
<!--eof-header logo and navigation display-->
</div>

Or am I missing something?

No. You are in exactly the right place. Though introducing branding doesn't necessarily require editing this file. Often it can be achieved by changing the elememts that it uses, such as the logo image (controlled by languages/english/header.php) or the background image for the header (controlled by your stylesheet.css file)

Yes, I see what you mean. To be truthful, I am just interested in using the digital downloads section of Zen cart and want to use my own shop front, headers and menus... Also, I have three images in the header two of them animated, That I share with my hobby site

In the interests of passing on the fruits of my labours:

<!--bof-branding display-->
<div id="logoWrapper">
     
     <!-- This deletes the header -->
     <!--<div id="logo"><?php echo '<a href="' . HTTP_SERVER . DIR_WS_CATALOG . '">' . zen_image($template->get_template_dir(HEADER_LOGO_IMAGE, DIR_WS_TEMPLATE, $current_page_base,'images'). '/' . HEADER_LOGO_IMAGE, HEADER_ALT_TEXT) . '</a>'; ?></div>-->
     
<!-- My code --> 
<?php include virtual("/my-site/zen-cart/includes/templates/gloss_black/common/myheader.shtml");?>
 
<!-- need this next for formatting page but set define tagline to "" in .../english/header.php-->    
<div id="taglineWrapper">

<!-- The rest does not seem to do much, so don't touch!-->
<?php
              if (SHOW_BANNERS_GROUP_SET2 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET2)) {
                if ($banner->RecordCount() > 0) {
?>
      <div id="bannerTwo" class="banners"><?php echo zen_display_banner('static', $banner);?></div>
<?php
                }
              }
?>
    </div>
</div>
<br class="clearBoth" />
<!--eof-branding display-->

You also need to change the css

/* don't want the header_back gif but need the rest */
#headerWrapper {
      background-color: #000000;
      
      /*background-image: url(../images/header_back.gif);*/
         
	margin: 0em;
	padding: 0em;
      }
/* End */

This seems to work for me and seems to give me complete control of menus, images and other content of the header.

Note, you have to use root relative addressing.

If anybody has any comments, or notices something bad, please let me know.