A Useful little 'fix' for layout.tpl

[michael +]

Well not so much a fix as a 'patch up' to cover Internet Explorers dismal rendering failure!

Current code with problem in red:

<!-- // Content // -->
<br /><br />
                        {* This includes the content portion *}
                        {include file=$body}
 <!-- // End Content // -->

Those two break tags were so frustrating! I suspect they were only ever placed there because IE sucked at rendering the original code, but by using them all other browsers had to pay the price in the form of some useless, dead whitespace beneath the search bar - right in the top, middle of your home page! I stripped them out and replaced them with a common enough hack as follows;

My new code with solution in red:

<!-- // Content // -->
<!--[if IE]>
    <br />
<![endif]-->
                        {* This includes the content portion *}
                        {include file=$body}
 <!-- // End Content // -->

[Eric Barnes +]

Thanks. Would you mind adding this to our bug tracker so I don't forget to fix this?
http://www.68classifieds.com/forums/.../bug-list.html

[Mike-N-Tosh +]

Michael,

Thanks for taking the time to post this thread. This is the type of community input that helps everyone and helps make the product better now and in the future. I added to your reputation for this post!

[seymourjames +]

Good one Michael. IE is just a royal pain as they say.

[michael +]

IE8 does seem to have upped it's game, the following amendment shoud take care of it;

<!--[if lte IE 7]>
<br /><br />
<![endif]-->

This way, the hack will only be applied to versions of IE that are less than or equal to 7.