allthegearnoidea.net

sirichmond

sirichmond

sirichmond

Associate
Joined
25 Apr 2007
Posts
139
Hello all,

A year ago I posted up for some feedback on my website http://www.allthegearnoidea.net and to be fair you guys slated it pretty badly for it being too cramped mainly and a bit loud design. So a year on I have just uploaded a new version so please check it out and give me some feedback again. I know its still not set out using CSS but I still have not had the time to get my head around it.

Many Thanks
Simon
 
Good:
It works
Simple design

Bad:
The header chequered effect. The text is fine, but what are the squares about?
Missing key mark-up (where are your h1 tags?)
JS rollover navigation that doesnt even need JS (CSS would do)

Very bad:
Tabled design! I know you explain this, but you still need to be told. Naughty!
 
<h1>Heading 1</h1>

< h1 >Heading 1< /h1 >


Useful tag for including headings of things. Used by Google in their algorithm as important page information.
 
sirichmond said:
Started adding H1 tag but as can be seen here http://www.allthegearnoidea.net/routes.html its making the surrounding black box which is just a table cell much bigger, how do I solve that.

Cheers
Si
Click to expand...

You want some CSS! Although I am going to give you the quick fix, which is an inline style.

So all H1 tags come with default properties, in this care it has a top and bottom margin applied, which is spacing out the black box.

So, your H1 tag wants to be along the lines of < h1 style="margin: 0" > < /h1 >

edit: infact, you have a class applied to your H1 tag! good man! now add margin: 0; to your stylesheet!

edit 2: hmm, ok, so your styles are included on that page in the header, not as an external stylesheet, but you are on your way...
 
Looks good, very snappy and fast. Easy to navigate without frustration.

Slighty off topic but the pictures make me laugh just because it rains so bloody much in UK. I've almost forgotten what it's like to have mud all over your face from a ride because it's poor etiquette to ride on the trails over here if it rains or if the trails are muddy.
 
Last edited:
Another thing while we are on the CSS subject, I would like to use 2 colours for my links, one standard blue/black for text on white other red/white as some text is on black background. Can I specify 2 in my CSS.
Cheers
Simon
 
Of course you can. Set the defaults and then for the ones you'd like different, set it again with either a class on the links, or if for example, these links are always in the same container, you can target all the links in the container.

ie:

Code: 
a { color: #fff;}
a.linkclass { color: #fff;}
div#containerName a { color: #fff;}
 
Tripnologist said:
ie:

Code: 
a { color: #fff;}
a.linkclass { color: #fff;}
div#containerName a { color: #fff;}
Click to expand...

Just as an explanation:
1) Sets all anchors to white as standard across everything (unless otherwise stated further on in the CSS document)
2) Sets all anchors with the class link, so < a class="linkclass" > to white
3) Sets all anchors within the div with ID containerName to white, so < div id="containerName" > < a > < /a > < /div >
 
sirichmond said:
can anyone tell me what it looks like on an iPhone, had a few people saying it dont display right. Cheers
Click to expand...

The 2 main columns are overlapping on the home page. The right column (Picture of the month etc.) is partially overlaid on the left column. The other pages look fine though.
 
You must log in or register to reply here.
Back
Top Bottom