[Webkit-unassigned] [Bug 6503] content property doesn't support quotes

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=6503


Carol Szabo <carol.szabo at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |carol.szabo at nokia.com




--- Comment #22 from Carol Szabo <carol.szabo at nokia.com>  2011-02-08 11:02:46 PST ---
(In reply to comment #21)
> > Source/WebCore/css/CSSStyleSelector.cpp:4599
> > +        HANDLE_INHERIT(quotes, Quotes)
> 
> Where is the handling of "initial"?
I do not see that initial is allowed for quotes. But maybe we should support it anyway.
> 
> > Source/WebCore/css/html.css:28
> > +    quotes: '"' '"' "'" "'";
> 
> This really shouldn't be necessary.  The CSS 2.1 specification says the initial value "depends on the user agent," which implies to me that the initial value is the quotes you want to use and not none.  Also, quotes should obviously work for XML documents, so let's just fix the initial value.
> 
> The initial value should probably just be a special keyword like -webkit-quotes.  Then your front end code can eventually use the current document language to pick the correct set of quotes (instead of hard-coding the quotes listed above).
I understand the part with using the quotes as per document language, but I am not sure how to implement that. An example of another feature using this would be helpful.
What I understand so far is that I could use a keyword (such as initial) instead of an actual list of quotes in html.css and in CSSStyleSelector::applyProperty, transform that keyword into a real list when creating the RenderStyle object for an element.
What I do not know is how to specify this keyword to the root element of any type of document so that it is inherited by all elements unless one element in the tree defines it otherwise, and then its children will inherit that redefined value.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.