Closed Bug 510463 Opened 15 years ago Closed 15 years ago

CSS font rules missed

Categories

(Core :: CSS Parsing and Computation, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 481502

People

(Reporter: cppiii, Unassigned)

Details

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 GTB5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 GTB5

A CSS rules for describing font was missed to render in Firefox 3.5.2:
body {
	font: normal 11px auto "Trebuchet MS", Verdana, Arial, Helvetica, sans-serif;
	color: #4f6b72;
	background: #E6EAE9;
}

It doesn't effort the fonts in body.

Reproducible: Always

Actual Results:  
Fonts weren't effort.

Expected Results:  
Fonts should follow the CSS rule to render.
jigsaw.w3.org validator gives a warning and converts this to:

body  {
font : normal 11px 'auto "Trebuchet MS"', Verdana, Arial, Helvetica, sans-serif;
color : #4f6b72;
background : #e6eae9;
}

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2 ID:20090729225027 gives error
Warning: Expected end of value but found '"Trebuchet MS"'.  Error in parsing value for 'font'.  Declaration dropped.
Component: General → Style System (CSS)
Product: Firefox → Core
QA Contact: general → style-system
Version: unspecified → Trunk
I guess => invalid from

"Similarly, quotation marks (both single and double), semicolons, exclamation marks, commas, and leading slashes within unquoted font family names must be escaped."

font: normal 11px auto \"Trebuchet MS\", Verdana, Arial, Helvetica, sans-serif;
It's nothing to do with the quotes, really; the "auto" is extra and makes the value syntactically incorrect.

The 'font' shorthand is special since font family names can take any keyword.  Therefore, font subproperties whose values take keywords must come before the size.  Then after the size, you can optionally have a slash and the line height, and after that, everything is the font-family.

So what's happening here is that you're trying to specify a font-family of:
  auto "Trebuchet MS", Verdana, Arial, Helvetica, sans-serif;

However, 'font-family' requires that each font in the comma-separated list either be quoted or not quoted.  We fixed this recently; we used to accept fonts that were partially quoted.

I think what you really want to do to fix this is remove the "auto".
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.