in the page
http://www.philipdusel.com/ppppp.html
the bottom border of the square is not showing in firefox.
I’m not familiar with this type of construct, but my guess is that the row collapses because the image it contains does not exist (http://www.philipdusel.com/templateImages/spacer.gif returns a 404). Try “spacer.gif” (remove “templateImages” from the src value) and see if that fixes your issue.
i could have sworn i checked that but then spent hours trying useless fixes…thanks!
*** WARNING *** I’m not going to sugarcoat this. Lip service “it’s fine” and silver bullet magic fixes are NOT going to help you any – this needs to be thrown away and started over using MODERN and VALID coding techniques. If I’m blunt, it’s because I’m trying to HELP.
Step one: Use VALID markup. Most likely with 90 validation errors -
You probably failed to close an element or twenty. Of course the problems run deeper when we look under the hood. Sixteen tables on a layout that even if still having one’s head up 1997’s backside to do tables for layout only warrants the use of TWO tables max. Here’s a tip, if your table is only going to have ONE TD, you shouldn’t be using a table. If it’s only going to have one TR filled with nothing but empty .gif’s… you shouldn’t be using tables or gif’s. (that’s padding and margin’s job!)…
Said bad coding decisions being the contributing factor to your 13k of markup to deliver 300 bytes of plaintext and six static images… easily four times the necessary markup for that!
… and that’s before we talk the lack of using thumbnails making the pageload take forever and the javascript for NOTHING.
The 90 validation errors are made even MORE amazing by the Tranny doctype. Is this a new site? If so it should be in STRICT. Transitional is for supporting old/outdated/half-assed coding techniques, NOT for building new websites. The mix of upper case, lack of quotes around values, and use of attributes and tags that are over a decade out of date make this EXACTLY the type of site I’m referring to when talking about people still writing HTML 3.2 and slapping a HTML 4 tranny doctype on it to call it “modern”.
Digging even deeper we have nonsensical heading orders, content cloaking, unclosed lists around non-list elements, table elements for tables that arean’t even OPEN, absolute URL’s for no good reason, no alt text on the images… I mean you don’t even actually CLOSE any of your IMG tags! (as in you are completely missing your closing greater-than signs!)
There is little reason for the markup on that page to be much more than:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html
xmlns="http://www.w3.org/1999/xhtml"
lang="en"
xml:lang="en"
><head>
<meta
http-equiv="Content-Type"
content="text/html; charset=utf-8"
/>
<meta
http-equiv="Content-Language"
content="en"
/>
<!--
Keywords meta should avoid redundancies and be limited to eight to ten words in 80 characters or less! Also it's called Keywords, NOT keyphrases!
not that keywords count for ANYTHING given the total lack of content for it to match on the page!
-->
<meta
name="keywords"
content="philip dusel,painter,sculpture,paintings,art,new york"
/>
<!--
Description meta exists to be the text shown on your SERP -- so describe the site!
-->
<meta
name="description"
content="Philip DuSel is a multitalented artist in the New York area."
/>
<link
type="text/css"
rel="stylesheet"
href="screen.css"
media="screen,projection,tv"
/>
<title>
Philip DuSel
</title>
</head><body>
<div id="pageWrapper">
<h1>Philip DuSel</h1>
<div id="contentWrapper">
<ul id="mainMenu">
<li>
<a href="/ppppp.html" class="current">PAINTING</a>
</li><li>
<a href="/sssss.html">sculpture</a>
</li><li>
<a href="/ddddd.html">digital art</a>
</li><li>
<a href="/resume.html">resume</a>
</li><li>
<a href="/contact.html">contact</a>
</li>
</ul>
<div id="content">
<ul class="breadcrumbs">
<li>&gt; Painting</li>
</ul>
<div class="pageSelect">
Page 1 of 3
<a
href="pages.php?content=gallery.php&page=2&navGallID=1&activeType="
class="nextPage"
>
>
<span><!-- image replacement --></span>
</a>
</div>
<table class="gallery2Up">
<tr>
<td>
<a href="/p1.html">
<img src="images/mymy7.jpg" alt="my my 7" />
</a>
</td><td>
<a href="/p2.html">
<img src="images/tree.jpg" alt="tree" />
</a>
</td>
</tr><tr>
<td>
<a href="/p3.html">
<img src="images/mymy3.gif" alt="my my 3" />
</a>
</td><td>
<a href="/p4.html">
<img src="images/bbccpainting3.gif" alt="bbcc Painting 3" />
</a>
</td>
</tr><tr>
<td>
<a href="/p5.html">
<img src="images/bbccpainting2.gif" alt="bbcc painting 2" />
</a>
</td><td>
<a href="/p6.html">
<img src="images/burgandy.gif" alt="burgandy" />
</a>
</td>
</tr>
</table>
<div class="pageSelect">
Page 1 of 3
<a
href="pages.php?content=gallery.php&page=2&navGallID=1&activeType="
class="nextPage"
>
>
<span><!-- image replacement --></span>
</a>
</div>
<!-- #content --></div>
<!-- #columnWrapper --></div>
<!-- #pageWrapper --></div>
</body></html>
Which is less than 3k WITH my over-the-top comments and leaving one table for layout in place. Everything else you are doing there, INCLUDING the hover effects should be handled in the CSS.
Oh, and I REALLY hope you aren’t going to flush what could be a decent gallery site down the toilet with that lightbox nonsense.
I’m a bit busy today, but maybe tomorrow if I remember I can belt out the CSS that would go with the above code.