mirror of
https://github.com/jquery/jquery-ui.git
synced 2026-04-20 03:02:41 -04:00
Merge pull request #353 from kborchers/bug_7043
Datepicker: Calculate the max number of rows necessary when displaying months. Fixes #7043 - Datepicker: Using multiple months always renders 6 rows of dates even if only 5 are needed
This commit is contained in:
2
themes/base/jquery.ui.datepicker.css
vendored
2
themes/base/jquery.ui.datepicker.css
vendored
@@ -39,7 +39,7 @@
|
||||
.ui-datepicker-multi .ui-datepicker-group-last .ui-datepicker-header { border-left-width:0; }
|
||||
.ui-datepicker-multi .ui-datepicker-group-middle .ui-datepicker-header { border-left-width:0; }
|
||||
.ui-datepicker-multi .ui-datepicker-buttonpane { clear:left; }
|
||||
.ui-datepicker-row-break { clear:both; width:100%; }
|
||||
.ui-datepicker-row-break { clear:both; width:100%; font-size:0em; }
|
||||
|
||||
/* RTL support */
|
||||
.ui-datepicker-rtl { direction: rtl; }
|
||||
|
||||
9
ui/jquery.ui.datepicker.js
vendored
9
ui/jquery.ui.datepicker.js
vendored
@@ -115,6 +115,9 @@ function Datepicker() {
|
||||
$.extend(Datepicker.prototype, {
|
||||
/* Class name added to elements to indicate already configured with a date picker. */
|
||||
markerClassName: 'hasDatepicker',
|
||||
|
||||
//Keep track of the maximum number of rows displayed (see #7043)
|
||||
maxRows: 4,
|
||||
|
||||
/* Debug logging (if enabled). */
|
||||
log: function () {
|
||||
@@ -691,6 +694,7 @@ $.extend(Datepicker.prototype, {
|
||||
/* Generate the date picker content. */
|
||||
_updateDatepicker: function(inst) {
|
||||
var self = this;
|
||||
self.maxRows = 4; //Reset the max number of rows being displayed (see #7043)
|
||||
var borders = $.datepicker._getBorders(inst.dpDiv);
|
||||
instActive = inst; // for delegate hover events
|
||||
inst.dpDiv.empty().append(this._generateHTML(inst));
|
||||
@@ -1480,6 +1484,7 @@ $.extend(Datepicker.prototype, {
|
||||
var html = '';
|
||||
for (var row = 0; row < numMonths[0]; row++) {
|
||||
var group = '';
|
||||
this.maxRows = 4;
|
||||
for (var col = 0; col < numMonths[1]; col++) {
|
||||
var selectedDate = this._daylightSavingAdjust(new Date(drawYear, drawMonth, inst.selectedDay));
|
||||
var cornerClass = ' ui-corner-all';
|
||||
@@ -1514,7 +1519,9 @@ $.extend(Datepicker.prototype, {
|
||||
if (drawYear == inst.selectedYear && drawMonth == inst.selectedMonth)
|
||||
inst.selectedDay = Math.min(inst.selectedDay, daysInMonth);
|
||||
var leadDays = (this._getFirstDayOfMonth(drawYear, drawMonth) - firstDay + 7) % 7;
|
||||
var numRows = (isMultiMonth ? 6 : Math.ceil((leadDays + daysInMonth) / 7)); // calculate the number of rows to generate
|
||||
var curRows = Math.ceil((leadDays + daysInMonth) / 7); // calculate the number of rows to generate
|
||||
var numRows = (isMultiMonth ? this.maxRows > curRows ? this.maxRows : curRows : curRows); //If multiple months, use the higher number of rows (see #7043)
|
||||
this.maxRows = numRows;
|
||||
var printDate = this._daylightSavingAdjust(new Date(drawYear, drawMonth, 1 - leadDays));
|
||||
for (var dRow = 0; dRow < numRows; dRow++) { // create date picker rows
|
||||
calender += '<tr>';
|
||||
|
||||
Reference in New Issue
Block a user