Change 20090522-maxcarlson-a by maxcarl...@bank on 2009-05-22 11:17:25 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean/my-apps
    for http://svn.openlaszlo.org/openlaszlo/trunk/my-apps

Summary: Turn text selection on in IE and Safari

Bugs Fixed: LPP-8200 - IE DHTML: selectable="true" does not work for <text>

Technical Reviewer: hminsky
QA Reviewer: (pending)
Doc Reviewer: (pending)

Details: DO NOT CHECK THIS IN!

This patch makes text selection work again in Safari. It will take some work before it's ready for prime time.

LzSprite - Don't turn on/off global mouse events - this interferes with selection somehow...

LzTextSprite - Use correct CSS preperty names for KHTML/webkit, set to null instead of an explicit value to clear to the default when selectable is on.

LzInputTextSprite - Avoid setting document.onselectstart handler - this also interferes with text selection.

Tests: Toggling on selection works in Safari
<canvas>
<button y="40" onclick="txt.setAttribute('selectable', ! txt.selectable)"/>
    <text name="txt">Hello Laszlo!</text>
</canvas>

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzInputTextSprite.js

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20090522-maxcarlson-a.tar

Reply via email to