Datepicker: Inline datepicker and showCurrentAtPos

[richelle]

Using an inline datepicker which displays several months and specifying showCurrentAtPos is not working correctly.

When you click on a date (doesn't matter which month) it moves all months back as many steps as you've specified for showCurrentAtPos.

Say, I have set numberOfMonths = 3 and showCurrentAtPos = 2, if I click any date the months displayed will move 2 months back (so if Aug, Sep, Oct are displayed, I click on any day, the months jump back so that now Jun, Jul, Aug are displayed). If showCurrentAtPos is set to 1, after clicking on a date, the calendars move back 1 month.

$('#dp').datepicker({numberOfMonths: 3, showCurrentAtPos: 2});

[sergik]

Hi all. I've fixed this issue in my project.

       /* Action for selecting a new month/year. */
	_selectMonthYear: function(id, select, period) {
		var target = $(id);
		var inst = this._getInst(target[0]);
		inst._selectingMonthYear = false;
		inst['selected' + (period == 'M' ? 'Month' : 'Year')] =
		inst['draw' + (period == 'M' ? 'Month' : 'Year')] =
			parseInt(select.options[select.selectedIndex].value,10);
		/*Fix line */
                if (period == 'Y'){inst['drawMonth'] = parseInt(inst['drawMonth'],10) + this._get(inst, 'showCurrentAtPos'); }
		this._notifyChange(inst);
		this._adjustDate(target);
	},

[lesc]

The fix from sergik above didn't work for me, as the _selectMonthYear method isn't fired when a date is selected (only from the dropdowns as far as I could tell). It appears that the calculation for moving the month location worked out every time the date is selected as well as when it's first drawn.

I've managed to get it working on mine by a quick (and probably dirty) fix, that passes a variable through to the _updateDatePicker, then _generateHTML methods so that it doesn't do the calculation on the clicking.

I would try and get this sorted cleanly, tested and forked on gitHub, but I'm a bit pressed for time, and this works for now. Hope it helps someone.

On line 892 change

this._updateDatepicker(inst);

to

this._updateDatepicker(inst, true);

Then on line 660 change

_updateDatepicker: function(inst) {

to

_updateDatepicker: function(inst, isClicked) {

Then on line 663 change

inst.dpDiv.empty().append(this._generateHTML(inst))

to

inst.dpDiv.empty().append(this._generateHTML(inst, isClicked))

Then on line 1353 change

_generateHTML: function(inst) {

to

_generateHTML: function(inst, isClicked) {

Then finally on line 1369 change

var drawMonth = inst.drawMonth - showCurrentAtPos;

to

var drawMonth = isClicked ? inst.drawMonth : (inst.drawMonth - showCurrentAtPos);

[amh-mw]

​https://github.com/jquery/jquery-ui/pull/571

[mikesherov]

#6733 is a duplicate of this ticket.

[tj.vantoll]

Test case against master - ​http://jsfiddle.net/tj_vantoll/guxUw/.

[tj.vantoll]

#9999 is a duplicate of this ticket.

[Scott González]

You're basically saying that you want showCurrentAtPos to be an init-only option and it shouldn't be considered after the datepicker has been rendered the first time. Is that what you're looking for?

I feel like there would need to be a different way to handle this than using showCurrentAtPos.