Thread: HTML DOJO
View Single Post
Old Oct 14th, 2012, 6:06:15 AM   #13
sandshrewz
穿山鼠
is a member of the Smogon Site Staffis an Artistis a Forum Moderatoris a Contributor to Smogon Mediais a Contributor to Smogonis a Battle Server Moderator
 
sandshrewz's Avatar
 
Moderator
Join Date: Oct 2010
Posts: 1,513
ASIA
Default

Quote:
Originally Posted by Fat Yonko7 View Post
Here's my version!

Hai


Jellicent covered the image and body tags well, follow what he said :) good use of <div> tag to center the image. I'll explain why h2 tags are used instead of h1. Go to Oglemi's article and press ctrl+u. Scroll down and you'll see that h1 tags are used for the article title. The [title] tag in the SCMS will wrap the title with h1 tags etc so that's where h1 is already used. h1 tags should never be used for anything :) and use [HTML][/HTML] instead of [CODE][/CODE] tags because the HTML tags will also color HTML elements~ :) another thing is the presentability of the HTML. Though it isn't a requirement, having a neat HTML is good as it makes editing and checking it easier, especially when edits need to be made on-site :p it's good practice to have spaces between <p> tags etc, wherever you deem fit. Oh one more since I spotted it last minute. DON'T USE <i> TAGS or <b> TAGS !_! It's spelt out in chaos's thread. Just use <em> and <strong> tags :)
Quote:
Originally Posted by Fat Redew View Post
...


yea you missed <em> tags too. It isn't compulsory to fill up the alt="" . Also you missed the alt="" /> in your last image tag :O Good job overall~
Quote:
Originally Posted by Fat Brutaka View Post
...


this was pretty well done. However you missed the italics used in the Introduction :x also the italics in the Togepi paragraph, you can just use <em> tags :) image tags should be <img src="/download/sprites/bw/113.png" alt="" /> instead of <img src="/download/sprites/bw/113.png" alt=""/>. Space needed after alt="" . And there's no need to use <br /> tags in this case. <p> tags will do as they're another paragraph. One thing to note though, that it's <br /> instead of <br/>. <div> tags can be used instead of <p style="...">. You used <p> probably to add an addditional space between the image and the last paragraph I guess? Doesn't really matter haha. Not much problems overall :)

Quote:
Originally Posted by Fat Furry View Post
Le Answer


You missed <em> tags as well too :O and don't put the image tags in new lines when they're on the same line :O title="" is not needed in this case. :p and yea spaces between <p> tags are good to have to :o you already have smog access so you should know what's the HTML standards~~ :p
Just a heads up. If Pokemon already has the accent in the article, you can leave it as Pokémon instead of Poke&eacute;mon :) good job everyone~ we'll see if Jellicent or Omicron has anything to add on :)
__________________
Member of the GP Team- VM for a check / stamp!
sandshrewz is offline   Reply With Quote