Bug 7157 – An asterisk in a CSS property name breaks CSS handling (Many layout problems at letras.terra.com.br)

• Component: CSS
• Status: RESOLVED FIXED

[Alexey Proskuryakov]

http://dalida.letras.terra.com.br/letras/60380

The page looks quite broken - not even sure if all the problems have the same root cause. Looks fine in Firefox.

Please note that the main text is actually broken (missing line breaks and badly encoded entities), so this is not part of the problem.

Created attachment 6388 [details]
Test case reduction (partly)

Part of the problem Safari has with this page is stars (*) used as prefixes to css property names. I'm not sure why they use this, but it doesn't validate. The part of the css that comes after these css properties seems to be ignored. Removing the star in the css of the attachment gives identical renderings from Firefox and Safari.

Removing all stars from the css of the real page removes most of the rendering errors in Safari. Curiously this breaks the rendering in Firefox.

Opera 9 agrees with Firefox in all cases.

Created attachment 6395 [details]
Test case reduction (partly), second try

I forgot an http refresh tag in the previous attachment. Nice during reduction, quite irritating and hitting on the server otherwise. Sorry for the spam.

[Alexey Proskuryakov]

Thank you for this really nice reduction!

Even if there are additional issues that make this page break, I think they will need separate bugs for tracking, and this bug should be devoted to the problem you found.

[Alexey Proskuryakov]

Created attachment 7337 [details]
proposed fix

Handle the case when the last rule is an error, and it doesn't end with a semicolon.

[Darin Adler]

Comment on attachment 7337 [details]
proposed fix

Looks fine, r=me.