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


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

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13857, comment 10

    initial v1  
    1 Strangely, it seems like the focus() call, like the appendix, isn't needed at all.    I tried removing it, and moving the other calls outside of the setTimeout(), and !ColorPalette seems to work fine on IE6, IE8, FF/win, and safari mac.
     1The 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 Both before and after my change, FF/mac has a rendering glitch on using the arrow keys to navigate, where multiple cells appear focused simultaneously.   It goes away by moving the mouse.
     3On 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.