[Webkit-unassigned] [Bug 7245] caret does not display at the end of some lines in contenteditable divs

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=7245


Graham.Dennis at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
Attachment #6564 is|0                           |1
           obsolete|                            |
   Attachment #6592|                            |review?(justin.garcia at apple.
               Flag|                            |com)




------- Comment #11 from Graham.Dennis at gmail.com  2006-02-18 15:33 PDT -------
Created an attachment (id=6592)
 --> (http://bugzilla.opendarwin.org/attachment.cgi?id=6592&action=view)
patch 2

New patch.

In this patch, instead of modifying the inlineStyleDecl of an element, I've
modified setContentEditable (called by parseMappedAttribute) to add in the
editing CSS properties. I've modified Frame::applyEditingStyleToElement (and
removeEditingStyle) to simply call setContentEditable on the element. This way
the inline style doesn't get blown away. Also, this way, the editing style can
be overridden by the inline style of the element.

This patch breaks many layout tests as it adds a space to the end of each line
in a contentEditable div. The vast majority of the problems are simply adding a
space and increasing the width of the rendered element (or its parents).
However, for two layout tests, this is not the case:
fast/overflow/overflow-focus-ring
fast/text/justified-text-rect

For the overflow-focus-ring test, the periods change from being on a single
line to being on many lines. This is caused by the "word-wrap: break-word"
style.

For the justified-text-rect test, the test checks that text is justified
correctly, however "-khtml-nbsp-mode: space" interferes with this. To be
honest, I don't know what "-khtml-nbsp-mode: space" does and whether the layout
test needs to be changed, the patch, or something else.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.