Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#8720 closed defect (fixed)

[regression]: FF2: themeTester.html Editor broken

Reported by: Douglas Hays Owned by: liucougar
Priority: high Milestone: 1.3
Component: Editor Version: 1.3.0b1
Keywords: Cc: Nathan Toone
Blocked By: Blocking:

Description

Run themeTester.html using Firefox 2 (I'm usng WinXP). Select the Textarea/Editor? tab and scroll down to the enabled Editor widget. You cannot focus or change it at all. This worked in 1.1.1. It stopped working with [14916].

Change History (4)

comment:1 Changed 11 years ago by Douglas Hays

This line:

this.document.designMode=(value?'off':'on');

is throwing an exception. The _delayedDisabled code seems to have changed the order in which this statement is being executed. It seems to work with Firefox 3.

comment:2 Changed 11 years ago by Douglas Hays

See #8727 for a patch. The exception used to be caught and then an onClick handler set designMode later. The _delayedDisabled code that waqs added in [14916] thwarted the try/catch. Also, the onclick handler connected in the catch did not allow TAB to establish focus.

comment:3 Changed 11 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [16801]) Fixes #8727, #8720. attr('disabled',val) throws an exception on hidden Editor widgets (eg themeTester tabs) so flag those and take care of things on the _onFocus handler (ie. no longer hidden). Also, for focusOnLoad=true, set focus after the Editor widgets have finished initializing to prevent the a disappearing input caret on FF2.

comment:4 Changed 11 years ago by Douglas Hays

Milestone: tbd1.3
Note: See TracTickets for help on using tickets.