Site Critique...yes, another...ya know ya love it...

axer

There are more problems. If your screen resolution is 800*600 you get the page like is attached. You really need to work on your layout and structure your divs better. I think that is why the portfolio page is messed up.

louie

Mirror wrote:

Ok guys, thanks all again! It's been updated a bit this afternoon, good or bad? Also, can anyone work out why the portfolio page extends to wide in IE7?? (possibly IE6- too, but I dont have any other versions to check)

Thanks!

Don't like it.
First: try to work on a fixed width layout (800px)
Second: used % to get the width of the inside divs, like 25% for the left and 75% for the main div. That will sort out your problem for screens with smaller resolution.
Third: those button style are so 70's (sorry).
Forth: Try the logo without the button look. It will look better. 70's as well.

axer

Mirror wrote:

Also, can anyone work out why the portfolio page extends to wide in IE7??

Your id called "main" has a width defined as 600px but the images in your portfolio page is wider than 600px.

I dont see the point in having your main div on the page set to 98% but yet having your content div set to a static 600px.

I'd follow louie's advice and stick to a fixed width layout.

Adam

Cheers guys, I'll sort the latest out tomorrow sometime. I know the divs and that got a bit all over the place, I had intended to just get the jist and then refine it all and make it nice and neat and cross browser compatible as much as possible, thanks for the advice though, I know what your saying!

I'm surprised about the opinion on the buttons, I didn't think they were really that bad...

Adam

Ok, I've spent the entire night (no, I haven't slept, thanks for asking!) swinging back and forth around different designs, and tiredness has prevailed, so I'm not sure whether I'm actually pleased with the result here or just too tired to care! I think I'm pleased though... if I could only come up with something nice for the navigation I'd be happy... any suggestions are welcome!

louie

First anytime you want us to look at the new changes try adding a link to the website in the post. I have to keep going back to the first post for it and this forum is kind of slow.

It looks a lot better, but for some reason in IE7 the text is bigger and it breaks apart the Portfolio page. In FF it looks grand.

I will try to use the logo directly in the source so you can make it a link back to the home page.

axer

It is a small bit better.

Why has the port id got a width of 600px when its contents are wider than that?

Would it not be better to use divs for each of the portfolio items than a table?

You put an image and text in the same link in the order of <img>TEXT but put the <br /> before the image. I think that is another thing that is causing the problem in IE7.

<a href="http://www.propertysaleorrent.ie/" target="_blank"><img src="http://www.cavanrock.com/images/psor_thumb.gif" alt="Property Sale Or Rent" class="port_img"><br>
Property Sale Or Rent</a>

instead of

<a href="http://www.propertysaleorrent.ie/" target="_blank"><br><img src="http://www.cavanrock.com/images/psor_thumb.gif" alt="Property Sale Or Rent" class="port_img">Property Sale Or Rent</a>

Adam

I need to stop working and start sleeping!

Have a munch on this anyhow: http://www.cavanrock.com/

More styles coming to a website near you!

axer

Definitely better.

You may want to drop the blue on word/phrases like "most competitive prices" etc. as blue suggests a link. Use the blue for links in the page and highlight the words using a different colour or just bold.

See if you can put an image or two in there somewhere as it looks a bit plain without any.

Overall a great improvement - well done.