History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: LPP-4200
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: P0 P0
Assignee: Unassigned
Reporter: André Bargull
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OpenLaszlo

Missing warnings in "LzBrowser.js" (DHTML)

Created: 25/Jun/07 12:49 PM   Updated: 29/Oct/07 09:50 AM
Component/s: Kernel - DHTML
Affects Version/s: 4.0.2
Fix Version/s: 4.0.5WaffleCone

Time Tracking:
Not Specified

Severity: Minor
Fixed in Change#: 6,137
Runtime: N/A
Flags: External
Fix in hand: False


 Description  « Hide
Print a warning (like: "This feature is currently not supported in DHTML-runtime") in the debugger for:
-LzBrowser#setClipboard(..)
-LzBrowser#isAAActive()
-LzBrowser#updateAccessibility()

---

Notes regarding clipboard-usage for DHTML:
Clipboard in IE: "http://msdn.microsoft.com/library/default.asp?url=/workshop/author/dhtml/reference/objects/clipboarddata.asp"
Clipboard in Firefox: "http://www.xulplanet.com/tutorials/mozsdk/clipboard.php" and "http://kb.mozillazine.org/Granting_JavaScript_access_to_the_clipboard"
Clipboard in Safari et al.: ???

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Max Carlson - 17/Aug/07 05:27 PM
Author: max
Date: 2007-08-17 17:14:47 -0700 (Fri, 17 Aug 2007)
New Revision: 6137

Modified:
   openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/views/LaszloCanvas.lzs
Log:
Should have been part of r6136...

Change 20070817-maxcarlson-L by maxcarlson@plastik on 2007-08-17 13:21:33 PDT
    in /Users/maxcarlson/openlaszlo/wafflecone
    for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone

Summary: Unify capability warnings across runtimes

New Features:

Bugs Fixed: LPP-4200 - Missing warnings in "LzBrowser.js" (DHTML)

Technical Reviewer: promanik
QA Reviewer: ben
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: LzInputText.lzs - Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

LzText.lzs - Only set advanced text methods and setters in construct() if capabilities.advancedtext is true. Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

LaszloView.lzs - Expose LzSprite.prototype.capabilities as view.capabilities. Add shared __warnCapability() method. Use this.capabilities instead of this.
sprite.capabilities and __warnCapability() instead of homespun warnings.


Tests: 'ant lztest' and smoketest pass.

Files:
M WEB-INF/lps/lfc/views/LzInputText.lzs
M WEB-INF/lps/lfc/views/LzText.lzs
M WEB-INF/lps/lfc/views/LaszloView.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070817-maxcarlson-L.tar



Modified: openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/views/LaszloCanvas.lzs
===================================================================
--- openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/views/LaszloCanvas.lzs 2007-08-18 00:13:09 UTC (rev 6136)
+++ openlaszlo/branches/wafflecone/WEB-INF/lps/lfc/views/LaszloCanvas.lzs 2007-08-18 00:14:47 UTC (rev 6137)
@@ -117,7 +117,7 @@
         if (args.width.charAt(args.width.length-1) == '%') {
             var percent = Number(args.width.substr(0, args.width.length-1));
             this.__canvaswidthratio = percent/100;
- if (this.sprite.capabilities.scalecanvastopercentage != true) {
+ if (this.capabilities.scalecanvastopercentage != true) {
                 // some runtimes already scale the viewport size, so take the reported size literally to avoid scaling twice
                 this.__canvaswidthratio = 1;
             }
@@ -136,7 +136,7 @@
         if (args.height.charAt(args.height.length-1) == '%') {
             var percent = Number(args.height.substr(0, args.height.length-1));
             this.__canvasheightratio = percent/100;
- if (this.sprite.capabilities.scalecanvastopercentage != true) {
+ if (this.capabilities.scalecanvastopercentage != true) {
                 // some runtimes already scale the viewport size, so take the reported size literally to avoid scaling twice
                 this.__canvasheightratio = 1;
             }


_______________________________________________
Laszlo-checkins mailing list
Laszlo-checkins@openlaszlo.org
http://www.openlaszlo.org/mailman/listinfo/laszlo-checkins

Steve O'Sullivan - 29/Oct/07 08:28 AM
Based upon the notes attached to the bug, I performed a search through these text files to verify that unwanted text was removed and that desired text was implemented.

LzInputText.lzs - Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities' found. '__warnCapability()' was implelmented.

LzText.lzs - Only set advanced text methods and setters in construct() if capabilities.advancedtext is true. Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities' found. 'this.capabilities' and '__warnCapability()' were implelmented.

LaszloView.lzs - Expose LzSprite.prototype.capabilities as view.capabilities. Add shared __warnCapability() method. Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities'. 'this.capabilities' was implemented. '__warnCapability()' was implemented.

Tests: 'ant lztest' and smoketest pass.

Found: ant lztest built successfully. 'Smoketest' is a text file not an lzx file.

Steve O'Sullivan - 29/Oct/07 08:35 AM
Based upon the notes attached to the bug, I performed a search through these text files to verify that unwanted text was removed and that desired text was implemented.

LzInputText.lzs - Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities' found. '__warnCapability()' was implelmented.

LzText.lzs - Only set advanced text methods and setters in construct() if capabilities.advancedtext is true. Use this.capabilities instead of this.sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities' found. 'this.capabilities' and '__warnCapability()' were implelmented.

LaszloView.lzs - Expose LzSprite.prototype.capabilities as view.capabilities. Add shared __warnCapability() method. Use this.capabilities instead of this.
sprite.capabilities and __warnCapability() instead of homespun warnings.

Found: No 'this.sprite.capabilities'. 'this.capabilities' was implemented. '__warnCapability()' was implemented.

Tests: 'ant lztest' and smoketest pass.

Found: ant lztest built successfully. 'Smoketest' is a text file not an lzx file.

Steve O'Sullivan - 29/Oct/07 09:50 AM
In addition to the text findings above, both tests: 'ant lztest' and smoketest passed. Bug closed.