You're right, this is a common use case. The right values can't be inferred, unfortunately. Perhaps the dhtmlKeyboardControl property should probably be passed into the embedDHTML call, the way appenddivid is now... I thought maybe folks would want to turn it on and off, which is why I left it in lzOptions...

This all got lost in the shuffle. Can you file a bug to clean this up and document it more?

Jim Grandy wrote:
Approved post-facto.

But I do have some design questions, since this seems like a fairly common use case. Is the need for a script setting lzOptions documented anywhere? Are the valid lzOptions properties documented? Could the right values for ServerRoot and dhtmlKeyboardControl be inferred from other inputs?

jim

On Mar 30, 2007, at 4:35 AM, Max Carlson wrote:

Change 20070330-maxcarlson-m by [EMAIL PROTECTED] on 2007-03-30 04:31:16 PDT
    in /Users/maxcarlson/openlaszlo/legals-clean
    for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: Fix mousewheel and keyboard navigation for documentation

New Features:

Bugs Fixed: LPP-3781 - documentation disables scrollwheel and navigation

Technical Reviewer: jgrandy
QA Reviewer: promanik
Doc Reviewer: jsundman

Documentation:

Release Notes:

Details: common-html.xsl - added lzOptions script with ServerRoot and dhtmlKeyboardControl: false directives. This prevents the LFC from snarfing keyboard control.


Tests: ant clean; ant doc; browse to http://localhost:8080/legals/docs/reference/LzAnimator.html and see that keyboard and mousewheel navigation work again.

Files:
M      docs/src/xsl/common-html.xsl

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070330-maxcarlson-m.tar


--
Regards,
Max Carlson
OpenLaszlo.org

Reply via email to