![]() | |
![]() |
| | Thread Tools | Display Modes |
#1
| |||
| |||
|
#2
| |||
| |||
|
|
Am redesigning the orchestra website to use PHP and be a little more interesting design-wise. The temporary location is http://www.opro.org/XXXnew%20redesign/index.php - I've put XXX in to not let the test directory be spidered, so remove that when you visit. |
#3
| |||
| |||
|
|
Am redesigning the orchestra website to use PHP and be a little more interesting design-wise. The temporary location is http://www.opro.org/XXXnew%20redesign/index.php - I've put XXX in to not let the test directory be spidered, so remove that when you visit. Not all links currently work, and there's tewaking here and there to do. Any comments welcome. |
|
Yuck div class="clear"> </div YuckČ Members Section To control freakish, css white-space would have sufficed here. |
|
h2>Welcome!</h2 Incorrect header usage, doesn't describe what follows. a href="prog.php" title="Past Programs">past concert programs</a Incorrect use of title attribute, same for most other links. |
#4
| |||
| |||
|
|
1. the cut-out photograph Both the image quality and the cut-outline leave much to be desired. IMO this badly needs a replacement, take a new one if you can't find anything suitable in the archives. |
|
2. the colour scheme. It just doesn't work. The strong blue clashes with the warm tones of the curtains in the photograph and the dull sandy background to the main content. Furthermore, the dull sandy background is just /too/ dull. Try brightening the background, keeping the hue but increasing saturation and mostly lightness. Instead of the strong blue, try something around the green tones with med-low saturation, and mid-brightness. |
|
PS. Just a thought but have you considered putting the calender below the menu? Having the menu to the left and the calender to the right leads to the eye jumping between the two with the content sitting rather uncomfortably in the middle. |
#5
| ||||||||||||
| ||||||||||||
|
|
Comment on html only (no link to stylesheet provided). |
|
Yuck div class="clear"> </div YuckČ |
I knew I was mucking it on those. But I'm not sure I understand exactly|
Members Section To control freakish, css white-space would have sufficed here. |
|
I don't see any need for the "related links" section, at least it should be positioned differently in the source, currently it precedes the content which makes little sense. |
|
h2>Welcome!</h2 Incorrect header usage, doesn't describe what follows. |
|
a href="prog.php" title="Past Programs">past concert programs</a Incorrect use of title attribute, same for most other links. |
|
Skip/jump to Jake'ism link nonsense. |
|
The presence/purpose of the "Western Mass Web Guide" button is a mystery. I've no idea what to expect when following the link. |
|
2 identical "Great Stories" links. |
|
Images are to pixilated. |
|
The top orchestra image fades to blue don't look good, and also fading the left and right edge to blue would look better. |
|
Imo there are 3 content sections on the index page: content, calendar, and site nav, all should be headed by a h2, currently there are more than 3 h2's used. |
#6
| |||
| |||
|
|
Neal wrote: The temporary location is http://www.opro.org/XXXnew%20redesign/index.php - I've put XXX in to not let the test directory be spidered, I suggest using robots.txt instead: User-agent: * Disallow: /new%20redesign/ |
#7
| |||||||||
| |||||||||
|
|
Yuck div class="clear"> </div YuckČ I knew I was mucking it on those. But I'm not sure I understand exactlywhat your objection is, it could be a number of things. |
|
The latter is present to make sure the div with the lower border has real content below the floats. |
|
I don't see any need for the "related links" section, at least it should be positioned differently in the source, currently it precedes the content which makes little sense. Where would be better, in your opinion? |
|
This could solve the issue with the <div> </div> after the local nav, if I can find a better spot. I don't want a branching "drop-down"-type menu, for sure. I kinda thought across the top was the most logical place for local navigation. |
|
h2>Welcome!</h2 Incorrect header usage, doesn't describe what follows. Dammit! |
|
a href="prog.php" title="Past Programs">past concert programs</a Incorrect use of title attribute, same for most other links. Agreed, but would you say the titles in the main navigation are OK? |
|
Skip/jump to Jake'ism link nonsense. Frankly I'm not sure either way. I'll allow the majority to rule on this. The presence/purpose of the "Western Mass Web Guide" button is a mystery. I've no idea what to expect when following the link. This is a recipricol for a listing in a local guide. Perhaps explanatory text is warranted. 2 identical "Great Stories" links. Yes, two separate concerts. |
|
Well, might move from the blue, as Ben isn't the first to not like the #000080 blue. Sadly, I kinda like it. |
|
Imo there are 3 content sections on the index page: content, calendar, and site nav, all should be headed by a h2, currently there are more than 3 h2's used. Wondering how I can logically lay it out. Perhaps the second main content section should be a H3, with better text for the H2 of the main content? |
#8
| |||
| |||
|
|
Neal wrote: On Tue, 26 Oct 2004 22:34:21 -0500, kchayka <usenet (AT) c-net (DOT) us> wrote: Neal wrote: The temporary location is http://www.opro.org/XXXnew%20redesign/index.php - I've put XXX in to not let the test directory be spidered, I suggest using robots.txt instead: User-agent: * Disallow: /new%20redesign/ 2) I've found that when you're using PHP the .htaccess isn't referred to for charset, wasn't sure of how successful such a disallow would work. .htaccess isn't involved at all here, just robots.txt. |
#9
| |||
| |||
|
|
div class="clear"> </div YuckČ I knew I was mucking it on those. But I'm not sure I understand exactlywhat your objection is, it could be a number of things. There are very few cases where the use of &nbps;'s is justified. First case: use css margin/padding, second case clearing floats should be done on the next block element. No need to inject bogus markup. The latter is present to make sure the div with the lower border has real content below the floats. Eh? |
#10
| |||
| |||
|
|
The latter is present to make sure the div with the lower border has real content below the floats. Eh? My guess is that he's encountered an IE bug. I have an otherwise spurious below a float on my pages because it was the only way I could get IE to behave. |
![]() |
| Thread Tools | |
| Display Modes | |
| |