Thread: HTML DOJO
View Single Post
Old Oct 11th, 2012, 4:00:53 PM   #5
Jellicent*
is 60% water
is a member of the Smogon Site Staffis a Super Moderatoris a Community Contributoris a Contributor to Smogon Mediais a Contributor to Smogon
 
Jellicent's Avatar
 
Super Moderator
Join Date: Sep 2011
Posts: 1,653
(80 PROOF)
Default

Normally, the <body> tag would be wrapped around the entire body of a page. You don't need to keep using <body> tags for each paragraph, as <p> already indicates that it's a new paragraph. For this one, <p> tags should also be wrapped around the images to give that added space. Finally, we don't use <body> tags at all (that I know of) for Smogon, as we have a SCMS tag for it, [page], which goes where <body> normally would and requires no [/page].

Also, your image tags are all broken >.>

<img src="http://www.smogon.com/download/sprites/bw/113.png"</img>

Should just look like:

<img src="/download/sprites/bw/113.png" alt="" />

There's no need to have a </img> tag, as it doesn't get wrapped around anything. Instead, you can end the tag with /> to make it complete.

Hope that helped ^.^
__________________
^.^
Jellicent is offline   Reply With Quote