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

Website Critque needed :Real Estate

1. When you roll over the active tab, there's no white arrow.

2. The Valid CSS button doesn't belong on a professional website like that.. unless you're trying to brag about all the technologies used (not a norm) and the other button isn't loading.

Otherwise, I like it. Where's the ajax though?
 
  1. I'll second the "no need for the Valid CSS button." Non-web developers won't know what that means. No need for it on a real estate site.
  2. The text in the Featured Developments area is on the small side.
  3. You're missing alt text on some of your images. You have the attribute, but nothing in it. Valid? Yes, Helpful? No.
  4. The text in the top menu and the search box keeps becoming bold, then unbold, and keeps switching and is distracting. Not sure what's causing it. Think it's the JavaScript doing the image transitions. (I'm using Firefox)
  5. In your accessibility links, the "Skip to 1st/2nd column" is uninformative for a user. They should describe the nature of that content or something more meaningful.
  6. Your XHTML almost validates. You may have noticed it has two errors (for the same problem), but may have been confused as to what the actual problem was. It took me a little bit to see it myself. The line is in the DWLayoutTable area. Inside the img tag the alt attribute is missing a space before it after the end " mark. Very easy fix. You're not far off from being able to validate against XHTML Strict either if you decide to strive for that.
  7. In the same section named above there is some empty anchor tags. I think they're there by mistake. Just something to check.
  8. I disagree with the above poster about the missing arrow on the "active" tab (yes, it doesn't show), I don't think it should be there, like implemented. It's the same way I implement my menu.
  9. An option, since the page shows signs of desiring accessibility. You can add an accesskey attribute to the login field. The accesskey would be applied to the input tag, but you can visually indicate the key on the label. I usually use and underline and bold on the character that accesses the accesskey. On your page using the 'u' (for Username) key would make sense. You could do the same with the search field as well.
  10. I don't think there's AJAX either. I see JavaScript, but that doesn't mean AJAX, though I didn't look into the script files to see if there are any server calls going on, which would make it AJAX.
  11. Contact Page: The middle of the page is VERY empty. I'd look at possible solutions for that. Also the label text on the fields is small. The page looks like it may be incomplete (since there's a sample image) so you may already be working on this.
I made a lot of comments, but most were easy changes. I like the overall look and feel so I concentrated on the underlying structure. I usually don't advise on some of the "deeper" things (e.g., accesskeys), but it seems like you're trying to address such things so thought you'd appreciate such comments.
 
Very nice. Nitpicks aside, it needs proof reading (from a quick skim of the About Us page).

PS I'm in the "No Arrow Necessary" camp, too.
 
angelwatt said:
  1. I'll second the "no need for the Valid CSS button." Non-web developers won't know what that means. No need for it on a real estate site.
  2. The text in the Featured Developments area is on the small side.
  3. You're missing alt text on some of your images. You have the attribute, but nothing in it. Valid? Yes, Helpful? No.
  4. The text in the top menu and the search box keeps becoming bold, then unbold, and keeps switching and is distracting. Not sure what's causing it. Think it's the JavaScript doing the image transitions. (I'm using Firefox)
  5. In your accessibility links, the "Skip to 1st/2nd column" is uninformative for a user. They should describe the nature of that content or something more meaningful.
  6. Your XHTML almost validates. You may have noticed it has two errors (for the same problem), but may have been confused as to what the actual problem was. It took me a little bit to see it myself. The line is in the DWLayoutTable area. Inside the img tag the alt attribute is missing a space before it after the end " mark. Very easy fix. You're not far off from being able to validate against XHTML Strict either if you decide to strive for that.
  7. In the same section named above there is some empty anchor tags. I think they're there by mistake. Just something to check.
  8. I disagree with the above poster about the missing arrow on the "active" tab (yes, it doesn't show), I don't think it should be there, like implemented. It's the same way I implement my menu.
  9. An option, since the page shows signs of desiring accessibility. You can add an accesskey attribute to the login field. The accesskey would be applied to the input tag, but you can visually indicate the key on the label. I usually use and underline and bold on the character that accesses the accesskey. On your page using the 'u' (for Username) key would make sense. You could do the same with the search field as well.
  10. I don't think there's AJAX either. I see JavaScript, but that doesn't mean AJAX, though I didn't look into the script files to see if there are any server calls going on, which would make it AJAX.
  11. Contact Page: The middle of the page is VERY empty. I'd look at possible solutions for that. Also the label text on the fields is small. The page looks like it may be incomplete (since there's a sample image) so you may already be working on this.
I made a lot of comments, but most were easy changes. I like the overall look and feel so I concentrated on the underlying structure. I usually don't advise on some of the "deeper" things (e.g., accesskeys), but it seems like you're trying to address such things so thought you'd appreciate such comments.
Click to expand...

I do appreciate the comments you've made, I'll be going through these changes now

many thanks to all your feedback!
 
You must log in or register to reply here.
Register on MacRumors! This sidebar will go away, and you'll see fewer ads.
Top Bottom
Back