Opened 11 years ago

Closed 11 years ago

#5052 closed bug (fixed)

datepicker: Set focus error in IE8 for disabled and hidden elements

Reported by: viraj.kanwade Owned by:
Priority: major Milestone: 1.8
Component: ui.datepicker Version: 1.7.2
Keywords: datepicker, IE8, disabled, display, none, hidden Cc:
Blocked by: Blocking:

Description

I added a textbox and attached a datepicker with a calendar button to it. To simply things for me, I disabled the textbox so that the user can enter date only through the calendar.

Everything worked fine till I tried it in IE8.

Message: Can't move focus to the control because it is invisible, not enabled, or of a type that does not accept the focus. Line: 573 Char: 5 Code: 0 URI: ​http://localhost/js/ui.datepicker.js

From the error, I figured out that IE8 has some issues setting focus to disabled or hidden elements.

A patch would be submitted in the next post.

Attachments (1)

ui.datepicker.ie8.disabled.displaynone.fix.patch​ (176 bytes) - added by viraj.kanwade 11 years ago.
Patch to fix the issue

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by viraj.kanwade

Attachment: ui.datepicker.ie8.disabled.displaynone.fix.patch​ added

Patch to fix the issue

comment:1 Changed 11 years ago by Scott González

Milestone: TBD → 1.8

With this patch, the type=hidden check is redundant with the addition of .is(':visible')

comment:2 Changed 11 years ago by kbwood

Resolution: → fixed
Status: new → closed

Fixed in r3728.

Note: See TracTickets for help on using tickets.