On 4/8/10 5:05 AM, P T Withington wrote:
Nit:

1) If we don't need this code any more, lets actually remove it.  Jira and svn 
leave enough of a paper trail for us to recover it.

Fair 'nuff.

Question:

1) What changed that we no longer need this?  I take it that the original test 
case in LPP-8160 still passes?

When we simplified the way inputtext and clickdivs work, the fix became unnecessary.

Approved technically.  But I somehow got magically upgraded to IE 8, so I'm 
relying on the QA reviewer to actually test the fix.

On 2010-04-08, at 01:37, Max Carlson wrote:

Change 20100407-maxcarlson-E by [email protected] on 2010-04-07 22:34:42 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix<html/>  mouse activation when an inputtext is focused in IE DHTML

Bugs Fixed: LPP-8884 - IE DHTML: focusing on inputext lose mouse events on 
iframe

Technical Reviewer: ptw
QA Reviewer: hminsky

Details: Back out fix for LPP-8160.  This is more voodoo that we don't appear 
to need anymore (whew!)

Tests: See LPP-8884 and LPP-8160

Files:
M       lps/includes/source/iframemanager.js

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20100407-maxcarlson-E.tar


Reply via email to