Become a MacRumors Supporter for $50/year with no ads, ability to filter front page stories, and private forums.

bntz313

macrumors 6502
Original poster
Jul 11, 2007
399
0
Okay, after posting my last portfolio that was a flash site, and getting a bunch of comments saying flash wasn't good. I went with a html, css.

let me know what you think, the code still needs some tweeking.

Remember, I'm still learning and all the content is there yet.

Portfolio
 
Very nice.
One good place to learn from is the W3C Validator - your code trips up on quite a few minor bits and pieces, which it would better for Goolability if it didn't.

Also on the search engine front (which doubles as a general check for accessible usability) there's somewhat of an absence of text on the site generally, which makes it hard for the bots to know what it is, what it's about and how to rank it.

And.. it would be preferable if the analytics code was inside the body rather than outside the html
 
I like the feel of it. Seems to match your style of work from the 2 items in your portfolio.

I agree, validation could be pretty easy to sort out - you have it mostly there. For validating with flash on the page see here.

...a flash site, and getting a bunch of comments saying flash wasn't good

And it's not that flash isn't good... but it's often used when just not necessary. Flash is generally an unnecessary complication on sites. There's no need to include it for no reason. For example, your flash header is (sorry to sound mean, it's not my intention) pointless. Having your head move up and the wording from the side adds no value to your site. This image could just as well be a gif or jpg.
 
ok so I removed the flash banner and I'm trying to fix all the validation, but can't seem to figure out were/what it is.
 
Righty,

HTML:
<a href="images/the_most.jpg" height="500" width="250" 
alt="The Most Beauitul" rel="lightbox[most_beautiful]" 
title="The Most Beautiful Women Ever!">
<img src="images/the_most_thumb.jpg" 
height="211" width="243" 
alt="The Most Beauitul"/></a>       
      
<a href="images/tcl_flyer.jpg" height="700" width="350" 
alt="The Most Beauitul" rel="lightbox[most_beautiful]" 
title="Three Chord Lords Poster">
<img src="images/TCL_thumb.jpg" 
height="211" width="243" 
alt="Three Chord Lords Poster"/></a>

In the above, you need to take the height and width out and use CSS to specify these by assigning a class to the anchor.

You should also take the alt text out of the anchor, and the title out of the image so you provide a title for your link and alternate text for your image.

HTML:
<!-- www.000webhost.com Analytics Code -->
<script type="text/javascript" src="http://analytics.hosting24.com/count.php"></script>
<noscript><a href="http://www.hosting24.com/"><img src="http://analytics.hosting24.com/count.php" alt="web hosting" /></a></noscript>
<!-- End Of Code -->

The above needs to go just before your </body> (I think - I've not dealt with this specific code).

Same on your portfolio page.

One other thing, not related to validation, the alt text on the header says 'header', not 'Design Portfolio'

Hopefully that will give you the green pass screen :) If not, I'll have another look.
 
Thanks man,

I love this site, there are some many talented people on here. Thanks why I try to help when I can.

I'm going to give a shot.
 
Register on MacRumors! This sidebar will go away, and you'll see fewer ads.