[Laszlo-checkins] r14263 - openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml

[hqm@openlaszlo.org]

Author: hqm
Date: 2009-06-29 20:12:45 -0700 (Mon, 29 Jun 2009)
New Revision: 14263

Modified:
   openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzBrowserKernel.lzs
   openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
Log:
Change 20090629-hqm-g by hqm at badtzmaru.home on 2009-06-29 23:10:42 EDT
    in /Users/hqm/openlaszlo/trunk6
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: fix for DHTML accessibility

New Features:

Bugs Fixed: LPP-8248

Technical Reviewer: max
QA Reviewer: antun
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:

+ LzBrowserKernel isAAActive cannot refer to canvas too early, or else
it will not have been bound as a global yet. We need to figure out how
the canvas accessibility flag should be referenced during the
initialization of the LFC, before the canvas has been created.

Tests:

smokecheck



Modified: openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzBrowserKernel.lzs
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzBrowserKernel.lzs	2009-06-30 02:23:38 UTC (rev 14262)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzBrowserKernel.lzs	2009-06-30 03:12:45 UTC (rev 14263)
@@ -181,7 +181,9 @@
   * @return: True if a screen reader is active and the Flash player is focused
   */
 static function isAAActive (){
-    return (canvas.sprite.capabilities.accessibility);
+    // Not yet implemented;
+    Debug.warn("LzBrowserKernel.isAAActive not yet fully implemented");
+    return false;
 }
 
 

Modified: openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
===================================================================
--- openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2009-06-30 02:23:38 UTC (rev 14262)
+++ openlaszlo/trunk/WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js	2009-06-30 03:12:45 UTC (rev 14263)
@@ -2414,7 +2414,8 @@
   * @param Boolean accessible
   */
 LzSprite.prototype.setAccessible = function(accessible) {
-    var a = LzBrowserKernel.isAAActive() && accessible;
+    // TODO [hqm 2009-06] also need to check LzBrowserKernel.isAAActive() when it is working
+    var a = accessible;
     LzSprite.__rootSprite.accessible = accessible;
 }