![]() | |
![]() |
| | Thread Tools | Display Modes |
#1
| |||
| |||
|
#2
| |||||
| |||||
|
|
Anyway, any comments are welcome: http://locusmeus.com/ |
|
span class="invisible"><a href="#content">skip navigation<br></a></span |
|
p class="fin" img src="fin.gif" alt="" /p |
|
div class="clearall"></div |
|
h1 span class="capital">L</span>ocus<span class="capital">M</span>eus.com /h1 |
#3
| |||||||||
| |||||||||
|
|
Els <els.aNOSPAM (AT) tiscali (DOT) nl> wrote: Anyway, any comments are welcome: http://locusmeus.com/ Newly authored documents should use a Strict doctype. |
|
The menu doesn't make sense sans css, menus should be marked up as an unordered list. |
|
You usage of <br>s in general is questionable, mid paragraph, or as used here: span class="invisible"><a href="#content">skip navigation<br></a></span |
|
Why not use a div? |
|
100 Kb of graphics is a bit much, I don't think you need such large jpegs (770x1000) to achieve that look. |
|
Linguistically "The title Locus Meus is Latin and means 'my place'. And that's what this site is all about." is somewhat odd, I'd suggest that this should be one sentence: "The title Locus Meus is Latin and means 'my place', and that's what this site is all about." |
|
p class="fin" img src="fin.gif" alt="" /p That's not a paragraph. |
|
div class="clearall"></div No need for this, just set clear:both on the div#bottom. |
|
h1 span class="capital">L</span>ocus<span class="capital">M</span>eus.com /h1 Slightly cleaner: h1.capital span{...} h1 class="capital" span>L</span>ocus<span>M</span>eus.com /h1 |
#4
| |||
| |||
|
|
A couple of weeks ago I asked for a site critique here, and the thing that was mentioned a lot, was that it lacked colour. So, I've put in some colour. cut / Anyway, any comments are welcome: http://locusmeus.com/ |
#5
| ||||||||
| ||||||||
|
|
Els wrote: A couple of weeks ago I asked for a site critique here, and the thing that was mentioned a lot, was that it lacked colour. So, I've put in some colour. cut / Anyway, any comments are welcome: http://locusmeus.com/ That looks very nice. I like it /much/ more than the previous version (although it was the winter theme I guess), very fresh and upbeat. |
|
You said you have problems with the nav links only being 'hot' over the text when using IE, have you tried a list for the navigation with the links inside the <li>s set to display: block? |
|
I might be tempted to put the menu descriptions as title attributes and use javascript to extract this and render the description box below the navigation. |
|
It doesn't make the site less accessible (might make it more accessible since those descriptions would be then be attached to the menu links themselves). |
|
Just a thought but you could get away with putting the navigation after the content in the markup and floating it left, that way content appears to load faster. |
|
I do know someone that can't read red text on white, so I'm just thinking that for her your links might disappear on hover! |
|
You might want to check those labels on the contact page, you've forgotten the ids! |
|
A very nice looking site! |
#6
| |||||
| |||||
|
|
Spartanicus wrote: Els <els.aNOSPAM (AT) tiscali (DOT) nl> wrote: [snip] |
|
Linguistically "The title Locus Meus is Latin and means 'my place'. And that's what this site is all about." is somewhat odd, I'd suggest that this should be one sentence: "The title Locus Meus is Latin and means 'my place', and that's what this site is all about." Good point. |
| [snip] |
|
div class="clearall"></div No need for this, just set clear:both on the div#bottom. Will check that out. |
#7
| |||
| |||
|
|
In message <40839d78$0$41754$5fc3050 (AT) dreader2 (DOT) news.tiscali.nl>, Els els.aNOSPAM (AT) tiscali (DOT) nl> writes Spartanicus wrote: Els <els.aNOSPAM (AT) tiscali (DOT) nl> wrote: Linguistically "The title Locus Meus is Latin and means 'my place'. And that's what this site is all about." is somewhat odd, I'd suggest that this should be one sentence: "The title Locus Meus is Latin and means 'my place', and that's what this site is all about." Good point. Leave it as it is -- it reads just fine. Starting that second sentence with 'And' provides just the right amount of emphasis. |
|
div class="clearall"></div No need for this, just set clear:both on the div#bottom. Will check that out. Leave it as it is. The advantage of doing it that way is that you can use it many times on the page (if you need to) without having the bother of incorporating the 'clear:both' in succeeding <div>s |
|
Anyway, it's a really nice looking site, and it reads well. |
#8
| |||||
| |||||
|
|
Andrew Urquhart wrote: [maybe try nested lists] Tried it just briefly, but if you say it's possible, I'll try it more thoroughly later. But I guess it would be just as difficult as Nico's solution, 'cause the links aren't all the same width. I don't want the submenu links to stick out from the rest. And I don't specify the width in pixels, but in em's, which makes it (afaik) impossible to get the submenu links the same width as the other links. (the font-size is 95% on the submenu links) Maybe I'll try a different approach to distinguish the sublinks from the rest, other than the indent and subsequent font-size. |
|
[use title and use accessible script to show it elsewhere] It doesn't make the site less accessible (might make it more accessible since those descriptions would be then be attached to the menu links themselves). How do you mean attached to the menu links themselves? Aren't they now? They're now part of the link, placed elsewhere using a span. |
|
[consider putting nav after content in markup] Is that a big difference? I'd prefer to have the menu (in the code) where it is now, but I never heard of the content loading faster if the menu comes later. Of course the page loads from top to bottom, so the first 8 lines will take their time first. But that isn't too much is it? |
|
[friend can't see red text on white] Is that a common sight-problem? |
|
[labels need ids] I'm afraid I haven't exactly forgotten them.. I have omitted them 'cause I don't know why they should be there. (Same goes for the labels, I just put them there 'cause William said I should, and I read in the specs how to.) Can you point me to an explanation of some sort about form labels and id's? (Not the specs, as apparently that didn't explain it well enough for me ;-) ) |
#9
| |||
| |||
|
|
jake wrote: [snip] |
|
Anyway, it's a really nice looking site, and it reads well. Thanks! :-) Now ...... just a few missing hyphens, slightly excessive use of commas, |
#10
| |||
| |||
|
|
In message <4083b569$0$41762$5fc3050 (AT) dreader2 (DOT) news.tiscali.nl>, Els els.aNOSPAM (AT) tiscali (DOT) nl> writes jake wrote: [snip] Anyway, it's a really nice looking site, and it reads well. Thanks! :-) Now ...... just a few missing hyphens, slightly excessive use of commas, and the odd mis-spelling to sort out ............. ;-) |
![]() |
| Thread Tools | |
| Display Modes | |
| |