Printable Version of Topic

Click here to view this topic in its original format

HTMLHelp Forums _ Site Review Requests _ Photography Website Review.

Posted by: GuessWho Jan 7 2010, 12:09 PM

Family Photography Website: http://www.photosbycrosby.com


I'm fairly new to these forums, but Ive already gotten some great help from you guys.
I know I still have some issues with the http://www.htmlhelp.com/cgi-bin/validate.cgi?url=http%3A%2F%2Fphotosbycrosby.com&warnings=yes&spider=yes, and recently found the http://jigsaw.w3.org/css-validator/validator?uri=photosbycrosby.com&profile=css21&usermedium=all&warning=1〈=en page (Could use some help with that one, Not sure what the correct way of doing that is... Obviously its the same error on my part aprox 66 times...)


Ive tested the site in both IE7 and FF 3.5.6
Please let me know if you notice something odd in any other browsers. As the site is mostly Photo based, I'm sure text based browsers (Lynx) will be all screwed up.. but that I can deal with.

Links are shown as White, when you mouse over available links they should go red.
a link to the current page being viewed should be gray on mouseover.
Pages that are not available (IE: Photographers pages who currently do not have anything available) will remain white on mouseover. I was going to have them go gray as well, but that seemed distracting more then anything.

Mousing over a photo should pop-up a medium sized thumbnail with some information beside it. Clicking that smaller thumbnail should open a Lightbox for the full sized image. Also, on photography pages, their is a small "?" button which should pop-up some 'help information'. yes, I know it is text embedded into the image at the moment, and I should probably change that.



Thoughts? Comments? Critiques? All more then welcome.
Thanks, Jon.

Posted by: pandy Jan 7 2010, 12:33 PM

All the CSS errors are of this type.

CODE
Value Error : width only 0 can be a length. You must put a unit after your number : 413


What the validator says. You should use a unit. Units can be omitted only when the value is zero.

Posted by: GuessWho Jan 7 2010, 02:00 PM

QUOTE(pandy @ Jan 7 2010, 10:33 AM) *

All the CSS errors are of this type.
CODE
Value Error : width only 0 can be a length. You must put a unit after your number : 413


What the validator says. You should use a unit. Units can be omitted only when the value is zero.



Ahhh... stupid mistake, Thanks. Quick fix..

All the warnings seem to pertain to lack of background colors, but I think its all viewed how I want it to be seen.

Posted by: pandy Jan 7 2010, 03:15 PM

QUOTE(GuessWho @ Jan 7 2010, 08:00 PM) *

All the warnings seem to pertain to lack of background colors, but I think its all viewed how I want it to be seen.


And vice versa. It's because if a user uses a user style sheet he could end up with your foreground color on his own background color (or the other way around) and they may not go well together and the text may not be contrasty enough to read. It's up to you if you think that's a realistic scenario or not. You don't have to use specific background colors everywhere. You can use 'inherit'.

Posted by: pandy Jan 7 2010, 03:26 PM

Cool idea BTW. A family of photographers. smile.gif

Posted by: GuessWho Jan 7 2010, 03:38 PM

Thanks again,
I put them in as inherit. A few of them are in front of multiple background colors, so I don't really want to set one specific color to them.


Thanks. yeah, Its pretty cool that we've all got the interest in common. somehow I got the responsibility of doing the web design even though I'd never done anything of the sort before. but It seems to be going well enough.

Any tips on those TR / TD errors? When I put them into tables (or Table rows, respectively) it really messes up the format.
(I think Ive got the Height ones fixed, well.. fixing those now anyways)

Posted by: pandy Jan 8 2010, 01:02 AM

My comment is CONGRATULATIONS, NO ERRORS! biggrin.gif

Posted by: stjepan Jan 9 2010, 06:41 AM

Histerius says: "Black design - we love!" smile.gif

Posted by: GuessWho Jan 9 2010, 06:03 PM

QUOTE(pandy @ Jan 7 2010, 11:02 PM) *

My comment is CONGRATULATIONS, NO ERRORS! biggrin.gif


Yay for no errors! hahah.. took a awhile but I got it! thanks for all the help.

QUOTE(stjepan @ Jan 9 2010, 04:41 AM) *

Histerius says: "Black design - we love!" smile.gif


Glad ya' like it. I wanted to go with something a little more professional than what I had seen with other Photography galleries. With having such a wide variety of photos, going with the earth tones didn't really work across the board for me. I had also tossed around some gradients in the background but didn't like how they looked, so I just kept it simple.

Posted by: GuessWho Jan 9 2010, 06:46 PM

Just noticed that for some reason, when in FF;
http://photosbycrosby.com/Contact.php
was being displayed slightly larger then any other page, thus the text embedded into images was blurry... All I had to do was go to my browsers "View" dropdown, --> "Zoom" --> "Reset" and it was fixed.. but I have no idea why it wasn't correct in the first place.

Anyone else notice this with that page? I'm assuming it was strictly something to do with my browser, but just wanted to check... (I really just need to remove the Text/Images...)

Posted by: GuessWho Jan 11 2010, 05:29 PM

So I was thinking I would simplify the pages by using a Div Content box...

The fact that it flashes (reloads the entire page) in IE bugs me.. is their a way around this so that it only loads the new content and not the entire page again (background image, etc..)


Thoughts?
Currently:
http://photosbycrosby.com/ContentTest/index.php?content=Home and http://photosbycrosby.com/ContentTest/index.php?content=Jon1 are loaded in... (all other links have been inactivated)

Posted by: GuessWho Jan 23 2010, 05:26 PM

Thanks '9 day website designing'!

Glad you like it. Its always great to hear that a couple website design company's like how Ive done things.


If anyone has suggestions on changes, let me know - I may not agree with them (hah), but I am always up for suggestions.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)