![]() | |
![]() |
| | Thread Tools | Display Modes |
#1
| |||
| |||
|
#2
| |||
| |||
|
|
Redoing a site for a local organization - please critique: http://test.glenmeadows.com Thank you, |
#3
| |||
| |||
|
#4
| |||
| |||
|
|
Redoing a site for a local organization - please critique: http://test.glenmeadows.com Thank you, |
#5
| |||
| |||
|
| http://test.glenmeadows.com The "birdlogo" is not centered over the menu. |
#6
| |||
| |||
|
|
aa wrote: Sorry about that 'aa' address. Was just trying out the slrn news client, and it kicked my boo-tay! |

#7
| |||
| |||
|
|
wayne wrote: Redoing a site for a local organization - please critique: http://test.glenmeadows.com Thank you, Hmm. I had your old glenmeadows.com site. At 'test'... Get rid of that animated fire gif! If you're going to label the images in the gallery at all, give them names useful to your visitors. Banjo_20050003.jpg doesn't convey much information to me at all, except that it has a banjo somewhere in it. You're using a poor thumbnailing program? |
#8
| |||
| |||
|
|
wayne wrote: http://test.glenmeadows.com The "birdlogo" is not centered over the menu. The colors in the "birdlogo" have no relation to the main color scheme. There are a couple of stray blotches in the "building" image on the home page. The upper left corner, and right center just above the tree. I would also crop out part of the sky and most of the grass. The BABA logo at the head of "Memberships" and "Photo Album" is at a different position than the rest of the pages. Make the text in the nav menu larger. You set it at 1em but that does not change anything since earlier you set the "content" font-size to 95%. So 1 em = 100% of the 95% of my desired font size. The link for the current page is not disabled in the nav menu. Why wrap <h1> in a "logo" div? Apply the styling directly. I agree with tm: The photo gallery would look better if it were fluid. It is easy to do, caption and all. And even works in IE. Overall the site looks good: simple, clean and consistent. |
#9
| |||
| |||
|
|
Thank you for your help. I changed the size and position of the bird logo to center it. Still have to work on colors. I cropped the building picture too. The dimensions set in the building's width and height attributes do |
|
The logo is wrapped in <h1> because there is supposed to be one on every page and it is supposed to be relevant to the web site (or so I've read). I had asked why the <h1> was wrapped in a div. It appears superfluous. |
|
As for disabling the current nav button, I use ssi includes and the menu is the same every time because it is the same file. Is there a method to programatically determine which page is displayed and disable that link? I use PHP <http://www.php.net/>. It generates the custom nav menu for |
#10
| |||
| |||
|
|
The dimensions set in the building's width and height attributes do match the image's dimensions; the aspect ratio is wrong for the image: 496/333 vs 748/354. To speed page loading reduce the image's actual size to match its target size. And use a lower JPEG quality factor to further reduce the size. It does not need to be a print quality image. |
|
The logo is wrapped in <h1> because there is supposed to be one on every page and it is supposed to be relevant to the web site (or so I've read). I had asked why the <h1> was wrapped in a div. It appears superfluous. IIRC, <h1> and the <title> should be rather similar, if not the same. As for disabling the current nav button, I use ssi includes and the menu is the same every time because it is the same file. Is there a method to programatically determine which page is displayed and disable that link? I use PHP <http://www.php.net/>. It generates the custom nav menu for each page. It makes maintenance much easier. And it could be used to create the gallery page(s) as well. Haven't had a chance to become familiar with PHP yet, but believe the |
![]() |
| Thread Tools | |
| Display Modes | |
| |