I figured I'd jump on the bandwagon and show you all my latest html masterpeice :unsure:. I don't have photoshop at the moment, and I need to learn php, but after that I should be able to make some real websites.
This layout was intended as an artist website. Obviously, it never got off the ground ^_^.
http://navras.t35.com (sorry about the popups :S. IE and firefox are blocking them for me, but one of my friends still gets them)
Comments etc would be great.
Page 1 of 1
Neon's Designs
#1
- Disciple
-
- Group: Members
- Posts: 1,422
- Joined: 28-February 04
- Gender:Male
- Location:Aussieland
Posted 02 September 2005 - 04:58 AM
#2
- Master Adept
-
- Group: Admin
- Posts: 2,521
- Joined: 15-February 04
- Gender:Male
- Location:Toronto, Ontario
Posted 02 September 2005 - 12:30 PM
It looks ok, but too much white methinks.
Here are some HTML comments:
No DOCTYPE or character set. This is a must. If you're using the XHTML document type, you need to add alt attributes to the image element and self close them, you need to have all styles in your sheet, etc.
The html for your flash is a little ugly. Don't use <EMBED> try this:
That is cleaner. You could throw in
Also, that site can be made easily without tables and there is at least 5 times as much markup due to <table...><tr><td></td></tr><.../table>.
You're not using semantics. <tt>N e w s</tt> is wrong. You could do a
<font> tags are bad!
Use ids when needed (like banner), etc. You also forgot to close your div.
Your javascript should be in the <head> section.
Here are some HTML comments:
No DOCTYPE or character set. This is a must. If you're using the XHTML document type, you need to add alt attributes to the image element and self close them, you need to have all styles in your sheet, etc.
The html for your flash is a little ugly. Don't use <EMBED> try this:
<object type="application/x-shockwave-flash" data="movie.swf" width="xxx" height="xxx"> <param name="movie" value="movie.swf" /> </object>
That is cleaner. You could throw in
codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=6,0,0,0"if you want for some browsers benefit.
Also, that site can be made easily without tables and there is at least 5 times as much markup due to <table...><tr><td></td></tr><.../table>.
You're not using semantics. <tt>N e w s</tt> is wrong. You could do a
h2.title {
font-family: monoscape, "Courier New";
font size: 24pt;
color: #000;
letter-spacing: 2px;
}
<h2 class="title">News</h2><font> tags are bad!
Use ids when needed (like banner), etc. You also forgot to close your div.
Your javascript should be in the <head> section.
#3
- Cool
-
- Group: Veterans
- Posts: 6,678
- Joined: 07-February 04
- Gender:Male
- Location:Room 101
- Interests:Metal, philosophy, percussion, literature, writing, theology, personal fitness, live music, tattoos.
-
AKA Agatio
Posted 02 September 2005 - 06:47 PM
Forking showoff ^_^
Love the incorporation of flash.
Problems: Code is a pile of shiz, fix it :unsure:
It doesn't flow.
Get a nice colour scheme, and maybe a bg image?
Has potential, but needs work ;)
Love the incorporation of flash.
Problems: Code is a pile of shiz, fix it :unsure:
It doesn't flow.
Get a nice colour scheme, and maybe a bg image?
Has potential, but needs work ;)
Page 1 of 1
