Changes between Version 1 and Version 2 of Ticket #13857, comment 10


Ignore:
Timestamp:
Sep 20, 2011, 10:29:37 AM (8 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13857, comment 10

    v1 v2  
    11The focus() call seems to only be needed on IE.   I tried removing the focus() call, and moving the other calls outside of the setTimeout(), and !ColorPalette seems to work fine on FF/win and safari mac.
    22
    3 On IE, the need for the setTimeout() is dubious.   Calendar's _onDayClick() also does a focus() call, but without a setTimeout().   !ColorPalette's setTimeout() was added along with the focus() call in [20794].  At that time !ColorPalette was composed of spans and divs.
     3On IE, the need for the setTimeout(), rather than a direct call to focus(), is dubious.   Calendar's _onDayClick() also does a focus() call, but without a setTimeout().   !ColorPalette's setTimeout() was added along with the focus() call in [20794], but I tried rolling back to [20794] and replacing the setTimeout() with a direct call to focus(), and the !ColorPalette.html DOH test runs without errors on both IE6 and IE8.