Finished site... tell me what's wrong with it.
Moderator: General Moderators
- Christopher
- Site Administrator
- Posts: 13596
- Joined: Wed Aug 25, 2004 7:54 pm
- Location: New York, NY, US
- feyd
- Neighborhood Spidermoddy
- Posts: 31559
- Joined: Mon Mar 29, 2004 3:24 pm
- Location: Bothell, Washington, USA
not valid xhtml: http://validator.w3.org/check?verbose=1 ... Findex.php
couldn't validate css due to page code error.
couldn't validate css due to page code error.
Yes... I realize this, but I wasn't sure if having them positioned to the left looked as good. I will post it with them on the left and tell me what ya think...arborint wrote:Seems fine. Here are a few comments:
- buttons on top bar are positioned far to the right of the rest of the page on my monitor
Fixed... I thinkarborint wrote: - the links in the content area of the home page don't work
I realize this, but it is sort of not avoidable. I think it is ok, because they do retain order, except that the one you clicked moves to the top, the rest fall back to where they were. I don't think I will change this because then I'd have to put the description box in the wrong spot if that makes any sense.arborint wrote: - the menu at the left is odd because the choices reorder so you can't remember what you have seen
Thanks!
Looks very good to me Ninja Space Goat. Very clear. it's immediately obvious what the website is about (ok, I cheat I saw it before...). One can directly see which units are available, etc. No marketing mambo jambo, just very clear information about what it is about and what action I can take. I agree with arborint about the left menu. But I can comprehend the reasoning behind it. One question though: what if there will be more units in the future? Are those added to the menu then? Because in that case it might be good to have some sort of way to keep track of which ones are visited and which not. A visitor browsing quickly through the options available would appreciate that I think.
Good idea... I will think about that.matthijs wrote: it might be good to have some sort of way to keep track of which ones are visited and which not. A visitor browsing quickly through the options available would appreciate that I think.
Will do... thanks.agtlewis wrote:preload the rollover image. There is a slight delay when mousing over for the first time. If you don't notice it clear your cache and try again.
One more thing to keep in mind (I notice this pretty often on other sites too). Add body background-color. I myself have set default background in my browser to light grey, since I hate reading plain text files on white background. Thus on your site the background I see is grey, since you did not define body background-color attribute.
- RobertGonzalez
- Site Administrator
- Posts: 14293
- Joined: Tue Sep 09, 2003 6:04 pm
- Location: Fremont, CA, USA
On your contact page, the second box down on the left shows empty (both regular state and rollover). It links to property=camino. Then below there is another box that shows camino and links to property=camino also. Just a heads up.
EDIT: Looking at the generated HTML, the condition I noted should not be happening. Here is the output HTML:
For some reason the Camino link seems to be carrying over into the next button below. Maybe it is my browser (FF 1.5.0.1).
EDIT: Looking at the generated HTML, the condition I noted should not be happening. Here is the output HTML:
Code: Select all
<div id="left_nav_bar">
<ul id="left_nav_menu">
<li><a href="property.php?property=shadowbrook" id="top"><p>Shadowbrook<br />Apartments</p></a></li>
<li><a href="property.php?property=camino"><p>Camino<br />Apartments</p></a></li>
<li><a href="property.php?property=nunnely"><p>Nunnely<br />Properties</p></a></li>
<li><a href="property.php?property=elliot"><p>Elliot<br />Court</p></a></li>
<li><a href="property.php?property=sawmill"><p>Sawmill<br />Villas</p></a></li>
<li id="below_menu"> <br/></li>
</ul>
</div>