[Webkit-unassigned] [Bug 10338] When contentEditable, cursor doesn't change to hand - Need WebPreference

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Wed Sep 6 06:42:54 PDT 2006


Graham.Dennis at gmail.com changed:

           What    |Removed                     |Added
  Attachment #10416|                            |review?
               Flag|                            |

------- Comment #1 from Graham.Dennis at gmail.com  2006-09-06 06:42 PDT -------
Created an attachment (id=10416)
 --> (http://bugzilla.opendarwin.org/attachment.cgi?id=10416&action=view)
patch for comment

I've had a stab at this, but I think my method is a bit hackish, and I'm
looking for comments. As such no changelog or testcase is included in this

To see why this is a bit hackish, consider the following sequence of events:
Say your current selection is either non-existent or outside of the editable
div with the link. Now you mouse down on the link. At this stage, the selection
is still what it was. Some time later (perhaps a really short time) you release
the mouse. At this point, the selection is now inside the editable div with the
link, and I'm not sure how to determine whether or not this link should be
followed based on information available at this point. 

To work around this, I've created an extra private variable in
HTMLAnchorElement, m_rootEditableElementForSelectionOnMouseDown, which is the
root editable element of the selection at the time of the mousedown event. This
variable is cleared on either a mouseout or a mouseclick event. Using the value
of this variable, during the mouseclick event, it is then possible to determine
whether or not the link should be followed.

Does anyone have any better ideas?

Also, this patch does not use WebPreferences as it is still an initial hack. Do
we want a preference for this, or is Dan's suggested behaviour OK to be

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.

More information about the webkit-unassigned mailing list