The Web Design Group

... Making the Web accessible to all.

Welcome Guest ( Log In | Register )

 
Reply to this topicStart new topic
> Photography Website Review., Crits and Comments more then welcome!
GuessWho
post Jan 7 2010, 12:09 PM
Post #1


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



Family Photography Website: 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 HTML Validation, and recently found the CSS Validation 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.

This post has been edited by GuessWho: Jan 7 2010, 12:12 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
pandy
post Jan 7 2010, 12:33 PM
Post #2


🌟Computer says no🌟
********

Group: WDG Moderators
Posts: 20,730
Joined: 9-August 06
Member No.: 6



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.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 7 2010, 02:00 PM
Post #3


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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.

This post has been edited by GuessWho: Jan 7 2010, 02:02 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
pandy
post Jan 7 2010, 03:15 PM
Post #4


🌟Computer says no🌟
********

Group: WDG Moderators
Posts: 20,730
Joined: 9-August 06
Member No.: 6



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'.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
pandy
post Jan 7 2010, 03:26 PM
Post #5


🌟Computer says no🌟
********

Group: WDG Moderators
Posts: 20,730
Joined: 9-August 06
Member No.: 6



Cool idea BTW. A family of photographers. smile.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 7 2010, 03:38 PM
Post #6


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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)

This post has been edited by GuessWho: Jan 7 2010, 03:48 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
pandy
post Jan 8 2010, 01:02 AM
Post #7


🌟Computer says no🌟
********

Group: WDG Moderators
Posts: 20,730
Joined: 9-August 06
Member No.: 6



My comment is CONGRATULATIONS, NO ERRORS! biggrin.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
stjepan
post Jan 9 2010, 06:41 AM
Post #8


Serious Coder
*****

Group: Members
Posts: 329
Joined: 15-October 06
From: zagreb, croatia
Member No.: 445



Histerius says: "Black design - we love!" smile.gif
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 9 2010, 06:03 PM
Post #9


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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.


This post has been edited by GuessWho: Jan 9 2010, 06:07 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 9 2010, 06:46 PM
Post #10


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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...)
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 11 2010, 05:29 PM
Post #11


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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:
Home (index) and Jon1 (really my page 4...) are loaded in... (all other links have been inactivated)

This post has been edited by GuessWho: Jan 11 2010, 05:33 PM
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post
GuessWho
post Jan 23 2010, 05:26 PM
Post #12


Novice
**

Group: Members
Posts: 26
Joined: 29-December 09
Member No.: 10,714



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.
User is offlinePM
Go to the top of the page
Toggle Multi-post QuotingQuote Post

Reply to this topicStart new topic
2 User(s) are reading this topic (2 Guests and 0 Anonymous Users)
0 Members:

 



- Lo-Fi Version Time is now: 23rd April 2024 - 01:21 AM