Skip to content

Commit 116acb4

Browse files
committed
Merge pull request #200 from kzys/trac-6704-2
Fixed #6704 - Display overflow when multiple datepickers have different numberOfMonths.
2 parents 511b86b + fc97ccd commit 116acb4

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

tests/unit/datepicker/datepicker_core.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,8 +116,16 @@ test('baseStructure', function() {
116116
ok(child.is('div.ui-datepicker-group') && child.is('div.ui-datepicker-group-last'), 'Structure multi [2] - second month division');
117117
child = dp.children(':eq(2)');
118118
ok(child.is('div.ui-datepicker-row-break'), 'Structure multi [2] - row break');
119+
ok(dp.is('.ui-datepicker-multi-2'), 'Structure multi [2] - multi-2');
119120
inp.datepicker('hide').datepicker('destroy');
120121

122+
// Multi-month 3
123+
inp = init('#inp', {numberOfMonths: 3});
124+
inp.focus();
125+
ok(dp.is('.ui-datepicker-multi-3'), 'Structure multi [3] - multi-3');
126+
ok(! dp.is('.ui-datepicker-multi-2'), 'Structure multi [3] - Trac #6704');
127+
inp.datepicker('hide').datepicker('destroy');
128+
121129
// Multi-month [2, 2]
122130
inp = init('#inp', {numberOfMonths: [2, 2]});
123131
inp.focus();

ui/jquery.ui.datepicker.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -685,10 +685,9 @@ $.extend(Datepicker.prototype, {
685685
var numMonths = this._getNumberOfMonths(inst);
686686
var cols = numMonths[1];
687687
var width = 17;
688+
inst.dpDiv.removeClass('ui-datepicker-multi-2 ui-datepicker-multi-3 ui-datepicker-multi-4').width('');
688689
if (cols > 1)
689690
inst.dpDiv.addClass('ui-datepicker-multi-' + cols).css('width', (width * cols) + 'em');
690-
else
691-
inst.dpDiv.removeClass('ui-datepicker-multi-2 ui-datepicker-multi-3 ui-datepicker-multi-4').width('');
692691
inst.dpDiv[(numMonths[0] != 1 || numMonths[1] != 1 ? 'add' : 'remove') +
693692
'Class']('ui-datepicker-multi');
694693
inst.dpDiv[(this._get(inst, 'isRTL') ? 'add' : 'remove') +

0 commit comments

Comments
 (0)