Please check my html for errors:

Soldato
Joined
3 Dec 2004
Posts
2,639
Hi guys,

I have a html template I use a lot that I have been adding/removing bits for ages. The output (when viewed in a browser) is just what I need but I think the coding behind it may have become a bit messey. Ive tried to tidy the coding up a bit but im not very good with html tables and everything falls apart. If someone could have a look and remove anything that is not needed (coding)it would be very much appricated.

Code:
<TABLE align=center border=2>



<TR vAlign=top>

<TD colSpan=2>

<CENTER><BR><FONT size=2><A href="http://feedback.ebay.co.uk/ws/eBayISAPI.dll?ViewFeedback&amp;userid=top.bargain&amp;ssPa   geName=STRK:ME:UFS" target=_blank><IMG alt=TopBargain src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/Top.BargainLogo.gif" border=0></A> <BR><BR>Purchase With Confidence. <A href="http://feedback.ebay.co.uk/ws/eBayISAPI.dll?ViewFeedback&amp;userid=top.bargain&amp;ssPa   geName=STRK:ME:UFS"><I>VIEW FEEDBACK.</I></A> A Trusted eBay Power Seller. <A href="http://members.ebay.co.uk/ws/eBayISAPI.dll?ViewUserPage&amp;userid=top.bargain"><IMG src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/aboutme-small.gif" border=0> </A><p></CENTER></FONT></TD></TR>

<TR vAlign=top>

<TD>

<P align=center><B>Description:</B></TD>

<TD rowSpan=6><IMG height=600 alt="Picture Name" src="URL" width=300> </TD></TR>

<TR vAlign=top>

<TD><BR>

<UL><FONT size=2>DESCRIPTION 

<P>

<LI>

<LI>

<LI>

<P>&nbsp;</P>

<LI>

<LI>
</UL></FONT></TD></TR>

<TR vAlign=top>

<TD>

<P align=center><B>Postage:</B> </P></TD></TR>

<TR vAlign=top>

<TD><BR>

<UL>

<LI><FONT size=2>Please Check Table At Bottom For UK Postal Rates 

<LI>Fast Delivery Via An Insured/Traceable Courier 

<LI>Please Click <A href="http://members.ebay.co.uk/ws/eBayISAPI.dll?ViewUserPage&amp;userid=top.bargain" border="0"><IMG src="http://pics.ebaystatic.com/aw/pics/uk/aboutme-small.gif" border=0></A> For Full Details.</LI></UL></FONT></TD></TR>

<TR vAlign=top>

<TD>

<P align=center><B>Payment Methods:</B> </P></TD></TR>

<TR vAlign=top>

<TD><BR>

<P align=center><A href="http://www.paypal.co.uk/uk" target=_blank><IMG style="WIDTH: 253px; HEIGHT: 56px" height=52 alt=TopBargain src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/PaypalLogo.gif" width=253 border=0></A></P>

<UL>

<LI><FONT size=2><I>Bank Transfer. 

<LI>Money/Postal Orders and Cheques Accepted.<FONT color=white>.................. </FONT>

<DIV></DIV></LI></UL></I></FONT></TD></TR></TABLE>
<DIV></DIV></LI></UL></I></FONT></TD></TR></TABLE>
 
I'm going through that and indenting it for you but I don't really get tables (in a general sense, I've only ever used them for a really small table and didn't need to modify them much so I don't know) but there are tags thrown really randomly about!

<p> <ul> and <fonts> that stick out of nowhere and don't close. but if i amend it i won't know if it works or not because I have no css, or view.

It's one of the most random use of the hypertext mark up i've seen! edit: the more I go through the less it makes sense :p :cool:
 
Last edited:
Huge thanks tom,

I did some editing through notepad + an editor which is what caused all the open tags etc. When I tried to tidy it up everything moved etc. I'll try sticking with just notepad from now on :)

Regarding the lowercase tags, i never knew that. I thought it was always uppercase :)
 
Back in the days of HTML, uppercase tags were quite popular. However, in the bright new dawn of XHTML, uppercase tags aren't allowed! :)
 
Here's a start, anyone else can edit and continue with this, I thought it'd be a 5minute boredom thing but I got intense, I've put up a doctype, seen as you 'kinda' need one of them to validate your html. But now you'll need to put your table knowledge into practise and fix it seen as before it wasn't set up correct.

Code:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head>

	<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1"/>

	<link rel="shortcut icon" href="img/favicon.ico" />

	<link rel="stylesheet" type="text/css" href="stylesheet.css" />

		<title>Top Bargain</title>

</head>
<body>
<table align=center border=2>
	<tr vAlign=top>
	<td colSpan=2>

	<center>
		<br />
			<font size=2><a href="http://feedback.ebay.co.uk/ws/eBayISAPI.dll?ViewFeedback&amp;userid=top.bargain&amp;ssPa   geName=StrK:ME:UFS" target="_blank"><img  src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/Top.BargainLogo.gif" alt="TopBargain" border="0" /></a>
			<br />
			<br />
			
			Purchase With Confidence. <a href="http://feedback.ebay.co.uk/ws/eBayISAPI.dll?ViewFeedback&amp;userid=top.bargain&amp;ssPa   geName=StrK:ME:UFS"><em>VIEW FEEDBACK.</em></a> 
			A Trusted eBay Power Seller. <a href="http://members.ebay.co.uk/ws/eBayISAPI.dll?ViewUserPage&amp;userid=top.bargain"><img src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/aboutme-small.gif" border="0"> </a><p>
			</font>
	</center>
	
	</td>
	</tr>

	<tr vAlign=top>
	<td>

		<h3 align=center>Description:</h3>
	</td>

	<td rowSpan=6>
	<img src="URL" width="300" height="600" alt="Picture Name" /> 
	
	</td>
	</tr>

	<tr vAlign=top>
	<td>
	
	<br />
		<ul>
			<font size=2><p>DESCRIPTION</p>
			<li></li>
			<li></li>
			<li></li>
			<li></li>
			<li></li>
			</font>
		</ul>
	</td>
	</tr>

	<tr vAlign=top>
	<td>
		<h3 align=center>Postage:</h3>
	</td>
	</tr>

	<tr vAlign=top>
	<td>
		<br />
		
		<ul><font size=2>
			<li>Please Check Table At Bottom For UK Postal Rates</li>
			<li>Fast Delivery Via An Insured/Traceable Courier</li>
			<li>Please Click <a href="http://members.ebay.co.uk/ws/eBayISAPI.dll?ViewUserPage&amp;userid=top.bargain" border="0"><img src="http://pics.ebaystatic.com/aw/pics/uk/aboutme-small.gif" border=0 /></a> For Full Details.</li>
		</font>
		</ul>
		
	
	</td>
	</tr>

	<tr vAlign=top>
	<td>

		<h3 align=center>Payment Methods:</h3>
	</td>
	</tr>

	<tr vAlign=top>
	<td>
	
	<br />

	<p align=center><a href="http://www.paypal.co.uk/uk" target=_blank><IMG style="WIDTH: 253px; HEIGHT: 56px" height=52 alt=TopBargain src="http://www.topbargain.pwp.blueyonder.co.uk/images/Template/PaypalLogo.gif" width=253 border=0></a></p>

		<ul><font size=2>
			<li><em>Bank Transfer.<em></li>
			<li>Money/Postal Orders and Cheques Accepted.<FONT color=white>.................. </font>
</body>	
</html>
I'm really not a fan of how you did stuff at the bottom with all the dots, infact I'll take that out, but it makes the table thinner. Get fixing. :p

There are tonnes of errors, but as I said, that was only a quick fix to get it looking neater and easier to fix yourself.
 
Last edited:
Wow huge thanks for your time and effort RandomTom, much appricated. Coding looking much better. I really do have to read up on HTML as a lot of it goes over my head.

lol @ the 'quick bodge' table extended I did:

Code:
 font color = "white" ...............

I could'nt/cant see away around that to make the table less thin. I'll look into it :)

If anyone else would like to add/suggest/edit your help is much appricated. Great start RandomTom, thanks!
 
Ive been tweaking RadomToms edit and I can see its a lot better. Only thing I cant fix thoug is the:

Code:
<li>Money/Postal Orders and Cheques Accepted.<FONT color=white>.................. </font>

Its used to make the table less thin and stop the border becoming to close to the words. I know someone said use CSS to work around it but I cant find out how to implement that.

Only other thing is the border is a bit thick around the headers (Description, Postage, Payment Methods), I ve tried changing the header number eg <h5> but although this changes font size the table dimensions are the same.

Thanks

Richard
 
Back
Top Bottom