/    Sign up×
Community /Pin to ProfileBookmark

Could i get some site feedback?

Now that i’m finally (mostly) done my site, i would love to have some people i don’t know tear it apart. Or just give me feedback.

[url]http://www.tylerstalman.com[/url]

to post a comment
Full-stack Developer

9 Comments(s)

Copy linkTweet thisAlerts:
@Sux0rZh_jc0rzOct 30.2003 — didnt check any of your code, but the site looks nice and loads fast. nice effects with the photo's btw. too bad there wasnt more content to browser through!
Copy linkTweet thisAlerts:
@spufiOct 30.2003 — http://validator.w3.org/check?uri=http%3A%2F%2Fwww.tylerstalman.com%2Fmain.php

There's your validation errors for your main page. Change the one <meta> tag so it's in HTML form and not XHTML form and that should clean up a number of errors. You seem to use some XHTML styled tags on a number of your pages. Either go with HTML, or XHTML.

http://www.alistapart.com/articles/flashsatay/

That link is going to help validate your <object> tag you use for Flash.

Since you have a external CSS file, there's no real reason to have stlyes defined within a tag. You can also get rid of the align attribute and define it in your external CSS file.

Granted me using a 1400X1050 resolution with a maxed out window isn't near the norm for what people view sites with, but that is a lot of white. Maybe using a ever so slight grey, or some other color for a background color around your main content area could help focus more towards your content. It just looks kind of sparse right now.

Your copyright statement could be more visible. It being bigger would be a good place to start.

The contact link probably should be in the regular menu. Why do you have a little form for contact and then you have a mailto option on your info page? I would just use a form for e-mail and not the mailto link. A form looks more professional to me.

I do like the site though.
Copy linkTweet thisAlerts:
@CupidDeLockeauthorOct 30.2003 — wow, that has to be the most informative helpful reply i could have gotten. thanks a lot ?
Copy linkTweet thisAlerts:
@CupidDeLockeauthorOct 31.2003 — i am a little confused as to where this xhtml reference is.
Copy linkTweet thisAlerts:
@spufiOct 31.2003 — <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />

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

<img src="images/girl11a.jpg" width=300 height=225 title="Click photo to enlarge">

The first two are in all of your <head> tags. XHTML requires all tags to have a closing tag. The problem with things like <meta>, <link>, and <img> tags, is there are no closing tags for them because they are considered empty tags. Empty as in all of the info is in the definition of the tag and nothing between the opening and closing tags. Anyway, change the <meta> and <link> tags to this.

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

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

The <img> tag is only partly correct for XHTML. I only checked the one page for the <img> tag, so I can only assume the others were done just like it. Here's the HTML version.

<img src="images/girl11a.jpg" width="300px" height="225px" alt="Name of girl incase the image doesn't display" title="Click photo to enlarge">

Technically an <img> tag in XHTML would look like this

<img src="images/girl11a.jpg" width="300px" height="225px" alt="Name of girl incase the image doesn't display" title="Click photo to enlarge" />
Copy linkTweet thisAlerts:
@pyroOct 31.2003 — [i]Originally posted by spufi [/i]

[B]Technically an <img> tag in XHTML would look like this



<img src="images/girl11a.jpg" width="300px" height="225px" alt="Name of girl incase the image doesn't display" title="Click photo to enlarge" /> [/B]
[/QUOTE]
Just one little thing to note with that: the width and height should be specified as nn for pixels and nn% for percentages (no px unless you are using styles). http://www.w3.org/TR/xhtml1/dtds.html#dtdentry_xhtml1-strict.dtd_Length
Copy linkTweet thisAlerts:
@spufiOct 31.2003 — Just note that if you do define them with "px" in XHTML, they will validate. At least in XHTML 1.1 they do.
Copy linkTweet thisAlerts:
@CupidDeLockeauthorOct 31.2003 — You guys rock my world, thanks. I fixed most of the things you mentioned, and took the others into consideration. When i get time i'm going to create a higher resolution version with larger thumbnails and photos.

However I'm still getting scads of errors on my embed tags, which i have no clue how to rectify.
Copy linkTweet thisAlerts:
@spufiOct 31.2003 — [i]Originally posted by CupidDeLocke [/i]

[B]However I'm still getting scads of errors on my embed tags, which i have no clue how to rectify. [/B][/QUOTE]


If you work your way through the Flash article I linked to, you will end up not using the <embed> tag at all. ?
×

Success!

Help @CupidDeLocke spread the word by sharing this article on Twitter...

Tweet This
Sign in
Forgot password?
Sign in with TwitchSign in with GithubCreate Account
about: ({
version: 0.1.9 BETA 6.17,
whats_new: community page,
up_next: more Davinci•003 tasks,
coming_soon: events calendar,
social: @webDeveloperHQ
});

legal: ({
terms: of use,
privacy: policy
});
changelog: (
version: 0.1.9,
notes: added community page

version: 0.1.8,
notes: added Davinci•003

version: 0.1.7,
notes: upvote answers to bounties

version: 0.1.6,
notes: article editor refresh
)...
recent_tips: (
tipper: @nearjob,
tipped: article
amount: 1000 SATS,

tipper: @meenaratha,
tipped: article
amount: 1000 SATS,

tipper: @meenaratha,
tipped: article
amount: 1000 SATS,
)...