Hi,

I'm new to css. I made this site http://www.nottinghamdrivingschool.biz and is the first time I've used css. More of an experimental version really to try and understand it.

After making it I realise that only one div id is supposed to be used and then multiple div class. I've put loooads of div id's one all of the pages.

So I am just wondering,

• is this site trash now and I pretty much need to rebuild?
• is it fine to just leave it as it is?
• is there an option to repair it?

Thanks

Rich

Honestly, it would work fine the way you have it in my opinion. If you've achieved the desired layout without issues, and the site loads fine, then I would say just leave it.

If you have multiple div ids that are pretty much identical to eachother, then what you could do is just re-assign them to one class you've set up in the CSS and reuse it multiple times. Make sense?

Thanks for that Reckoner.
Wasn't sure if it was some fatal mistake or something.

I think what you said makes sense...right noob on css.
I will study it more taking into account your advice.

they are not "div ids"

the ID attribute can be applied to every HTML element.

And yes an ID can ONLY be used once on a page.

You CAN however apply multiple classes to an element, with each classname seperated by a space.

you also need to learn about specificity and inheritance for where you have things like

<div class="footer"><a class="footer" ... >text</a></div>

The anchor element would inherit some properties from the parent and any property you want to change you can target by using descendant selectors.

.footer a{
/* Rule set*/
}

in the style sheet.
So the classname applied to the anchor is superfluous

Your ID's are fine, as long as they are UNIQUE on a page.

However, I make these suggestions:

1 - put your text inside proper <p></p>, don't put text inside unnecessary spans. Style the <p> tag overall, then use classes or spans only where needed.

2- style your links in the same manner

Well that's given me a lot to think about.
Thanks very much for your time!