Style code trouble!

Red One · #1 02-15-2011, 08:23 PM

I have two pages that I recently validated through W3C and had the validator clean up the errors for me. Those two pages were uploaded again to the main site but the background color became black instead of the original color of purple. I tried inserting just regular bg color code but that didn't work.

<style type="text/css">
/*<![CDATA[*/
body.c8 {background-color:#37007b}
p.c7 {margin-left:-325px;}
div.c6 {margin-top:-50px; margin-left:-50px;}
img.c5 {margin-top:-301px; margin-right:15px;}
img.c4 {margin-left:-324px;}
p.c3 {float: right; width: 50%; text-align: center;}
p.c2 {float: left; width: 50%; text-align: center;}
img.c1 {margin-right:330px}
/*]]>*/
</style>

This is the style type that contains the error. Note that the bg color is set to the custom value. This is the other style for the other page I happened to correct( I attempted copy and paste but had no luck either):

<style type="text/css">
/*<![CDATA[*/
body {background-color:#37007b}
p.c5 {margin-left:-325px;}
div.c4 {margin-top:-50px; margin-left:-50px;}
div.c3 {text-align: center}
p.c2 {font-size: 144%}
img.c1 {margin-right:330px}
/*]]>*/
</style>

If anyone has an idea of what could be wrong, I would greatly apprecitate the help!

Dan Zollman · #2 02-15-2011, 08:51 PM

Could you clarify which block of code is working and which isn't?

Try comparing the selectors in each blog. For example, the first one has styles for

Code:

body.c8

which means they will apply to the body tag only if you have

Code:

<body class="c8">

in the HTML. In the second block, the selector is just "body", meaning that it will apply
to the body tag regardless of its classes.

If you compare the selectors as well as the classes used in the HTML itself, that might explain the results you're getting.

perlgerl · #3 02-16-2011, 09:18 PM

Did you miss the semi-colon? eg

body.c8 {background-color:#37007b;}

Cecil · #4 02-16-2011, 09:38 PM

Quote:

Originally Posted by perlgerl

Did you miss the semi-colon? eg

body.c8 {background-color:#37007b;}

Normally, you don't need the semicolon for the last one. You only need it if you are separating multiple styles for the same thing.

aechmtwash11 · #5 02-16-2011, 09:50 PM

i would say add semi colons to everything and remove the /*<![CDATA[*/ & /*]]>*/ (What is that stuff even for?)

Dan Zollman · #6 02-17-2011, 01:20 AM

Quote:

Originally Posted by aechmtwash11

i would say add semi colons to everything and remove the /*<![CDATA[*/ & /*]]>*/ (What is that stuff even for?)

Yup, the CDATA stuff is not needed--but in this case, both lines are /* commented out */ so they won't impact the CSS.

And Cecil is correct; the last semicolon in each block is optional.

Thread Tools
Show Printable Version Email this Page
Display Modes	Rate This Thread
Switch to Linear Mode Hybrid Mode Switch to Threaded Mode	Rate This Thread: