I wanna be critiqued!

[LaurenMarie]

What kind of errors does he have, Tommy? How can he fix them? Any suggestions or advice?

[tommylogic]

if you click the link I provided, it will tell you, but.. here's the skinny..



Pretty basic fixes.. OPACITY isn't valid yet. Hopefully it will be supported in CSS 3.0

[Dajo]

Hey all, ok I am workin everything out in photoshop right now, I think I know my main problem, and all of you will wonder why the hell would this guy even use this! well yes I was using IWEB. now Im teachin myself dreamweaver. Of course Im liking it alot better. Im not going to advertise the page to anyone as of yet until I have the main finished product. I will be showing you guys what I come up with in photoshop and then go from their thanks to all your help! I am more of the graphic designer than the web designer. I have also changed my logo about a billion times and am closing in on the final version. THANKS AGAIN FOR ALL THE HONEST CRITIQUE. I almost changed my company name I was getting so freakin frustrated with coming up with IDEAS for my logo. However I have allready bought the domain obviously and that wouldve been a total waste of money and time.

[pa007]

A lot has been said about the design already so I won't repeat the words of other but there are serious issues with your code. You've made the big step of using CSS to create your site rather than tables which is great but there are still a few issues that you'll want to address now.

I think you need to take a step back and look at how you've developed your iste (code-wise). I'll use the home page as an example. I would probably use 4 or perhaps 5 divs depending on whether to not there would be a footer. You seem to have used quite a few. It's a big task at the outset but plan it out, the structure of the site (header, nav, left content, right content, etc.) should be the structure of your html. Try and keep your code down to a minimum. Bloated code is bad for accessibility and SEO.

Next point is your CSS. It should be linked either by using the link element or the @import method. Both are great and have their advantages. You should be separating style form content. Any behaviour should also be in an external file. Think of it this way, it would be possible with clean code and external CSS to change your whole design by altering a single file (providing you only have one .css file, sometimes it's good to have a few different ones to make it easier to navigate your css at a later date).

Also you are putting other presentational elements into the html when they should be in the CSS. Your logo/header is really a visual enhancement of your company name. So you would be better off having:

Imagefix Solutions - Web and graphic Design

or perhaps, fr better SEO:

Imagefix Solutions - Web, graphic Design

Then using a CSS image replacement technique you can swap out this extremely accessible and keyword rich text for your header image or logo. The same for your navigation menu. It should be a nice unordered list, pure text. All navs should be. Then, if it is needed, you can swap out the text for images. In this case I'm not sure you need images but that's up to you.

Alos you need to use all of the available elements correctly. Think about the meaning of your content. A div element is meant to decide things and is semantically neutral, you've used a few too many here. Remeber with CSS you can style anything, however you want. Paragraph text should be in a

element, your heading should be in a

or

element (depends whether or not you use a h1 for you company info/header). You see, there are lots of different elements available (such as for contact information), make good use out of them. It's good practice and can aid accessibility and help increase you search rankings.

Final note, use a strict doctype. It doesn't matter whether you use HTML or XHTML they are both perfectly valid, what matters is that you use the doctype that helps make sure your code is the best it can be. There are other reasons for using a strict doctype that I won't go into right now.

There is more to this web standards malarkey than tableless layouts but it's not so bad when you sus it.

It's a lot to take in, I know, but if you start thinking about these things now it'll come together quicker than you might imagine.

Your layout is nice you just need a design concept. Try a metaphor, they're used a lot in advertsing and communicate well to users.viewers. As it is it took me a littlw hile to work out what you did and that isn't ideal. If you only achieve one thing, make sure a user knows what you do withing a few seconds. Tag lines can help but a stronger brand would be more useful (oop, I said I wouldn't talk about the design).

Apologies for the long post, I'm new around here and here I am taking up precious page space.

Pete. :)

[jkinz]

Under your portfolio the text is off to the right and is unreadable. If you fix the errors and double check all of your links, etc. for broken links (there are a few in the portfolio whre the image won't dispaly) you should be good.

[matt2364]

Looks pretty good, I would try to do a little something to the opening page, looks pretty blan, but I did end up reading the page, which I probably would not usually do, so that is a good thing.