Skip to main content

Search and Top Navigation

#5984 closed bug (fixed)

Opened August 24, 2010 05:07AM UTC

Closed September 20, 2010 01:51PM UTC

Last modified November 19, 2010 06:26PM UTC

Datepicker showing multiple months highlights selected day in all of them

Reported by: kbwood Owned by:
Priority: major Milestone: 1.8.6
Component: ui.datepicker Version: 1.8.4
Keywords: Cc:
Blocked by: Blocking:
Description

From forums: http://forum.jquery.com/topic/datepicker-showing-multiple-months-highlights-selected-day-in-all-of-them.

If you configure a datepicker to show multiple months, and the field it's tied to has a date entered, when the picker opens it highlights that day of the month in all displayed months. You can see this happen in the 'Display multiple months' demo, by clicking in the field to open the picker, choosing a date, then opening the picker again. Whatever day you picked is highlighted in all months, so there's no way to tell which one is actually selected.

Revert the change made here (jquery.ui.datepicker.js, line 1496):

            (printDate.getTime() == selectedDate.getTime() ? ' ui-state-active' : '') + // highlight selected day

to this:

            (printDate.getTime() == currentDate.getTime() ? ' ui-state-active' : '') + // highlight selected day
Attachments (1)
Change History (4)

Changed September 20, 2010 01:51PM UTC by scottgonzalez comment:1

milestone: TBD1.9
resolution: → fixed
status: newclosed

Fixed in a936eb3.

Changed September 30, 2010 09:55AM UTC by ergec comment:2

If you are using minified & combined version of UI downloaded through ThemeRoller (jquery-ui-1.8.5.custom.min.js), you won't be able to find the line mentioned above.

Instead, try to change this (jquery-ui-1.8.5.custom.min.js, line 534):

q.getTime()==J.getTime()?" ui-state-active":""

into this:

q.getTime()==u.getTime()?" ui-state-active":""

Changed October 25, 2010 06:53PM UTC by scottgonzalez comment:3

milestone: 1.91.8.6

Changed November 19, 2010 06:26PM UTC by Lim Chee Aun comment:4

Datepicker: Hightlight the curerent date, not the selected date. Fixes #5984 - Datepicker showing multiple months highlights selected day in all of them

Changeset: a936eb3c0982293940a3f3135fc85178bf17534d