(I am checking this change in.)

Change philip-20110711-nKh by philip@PHILIP-I7 on 2011-07-11 15:49:37 EDT
    in /cygdrive/c/clients/laszlo/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Fix backspace with IE by focusing on RTE component

New Features:

Bugs Fixed: LPP-9965

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

Documentation:

Release Notes:

Overview:
Trying to get dijit.Editor to give up focus does not work in IE. I added a new rte component attribute, focusonload. When set to true, the editor will receive focus. In IE this means the backspace key does nothing because focus is in a text element. For IE7, I had to listen to a blur event in order to get the focus from the rte to the flash input text.

You should modify the rte components to set focusonload="true".


Details:
rte.lzx
- New focusonload attribute. This defaults to false. Make it true so the rte editor will get focus when it is loaded. - Removed previous focus/blur code as it has problems in certain browsers.

rtemanager.lzx
  - focusonload support.
  - Listen to blur events to move focus for IE (especially IE7).
- When the editor is loaded, a callback method called rte_ready() will be called in both the application wrapper file and rte wrapper file. This can be used to complete initialization when the rte is loaded. This is not currently used.

Tests:
Test case in 9965, but one modification is needed. To each RTE component add the attribute,
  focusonload="true"


Files:
M       lps/components/extensions/rte.lzx
M       lps/includes/source/rtemanager.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/philip-20110711-nKh.tar

Reply via email to