[Webkit-unassigned] [Bug 27988] word-spacing style seems not to be honored

[bugzilla-daemon at webkit.org]

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


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34293|0                           |1
        is obsolete|                            |
  Attachment #34311|                            |review?
               Flag|                            |




--- Comment #6 from Carol Szabo <carol.szabo at nokia.com>  2009-08-07 12:02:22 PDT ---
Created an attachment (id=34311)
 --> (https://bugs.webkit.org/attachment.cgi?id=34311)
Fixed indent size.

Fixed indent size per guidelines.
The reason why I chose this fix is because I wanted to minimize differences
between the Qt implementation and the other implementations.
On most platforms letter spacing and word spacing is not handled like on Qt by
the platform font; hence most of the platform independent infrastructure is set
up for specifying platform fonts independent of word and letter spacing.
Furthermore the letter and word spacing attributes can be set on the platform
independent Font class via accessors that do not notify anybody such that the
platform dependent dependent Font framework is unaware of changes to those
properties hence the items in various caches cannot be updated.
The getQtFont function is only called from the font() function that I changed.
If moving my code to that function and adding to the function the two required
extra parameters makes everybody happy, I will do that but I did not see the
extra value (perhaps I do not understand the architecture enough). I tried to
keep SimpleFontData as consistent as possible between platforms. I hate
variations in interface based on platform, and I did not want to reinforce the
behavior already present in SimpleFontData.

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