Use proper class names when options.weeks and options.className are both set.

When options.weeks is true, and options.className is set, the result is
that both are concatenated together without a space to separate them.

For future maintainability, I decided to split out the modifications to
the datetimepicker element into separate modifications to more cleanly
show intent.
This commit is contained in:
Sjoerd Job Postmus 2015-05-07 22:17:21 +02:00
parent e864c5a260
commit 3262fbdcb5

View File

@ -735,7 +735,7 @@
};
createDateTimePicker = function (input) {
var datetimepicker = $('<div ' + (options.id ? 'id="' + options.id + '"' : '') + ' ' + (options.style ? 'style="' + options.style + '"' : '') + ' class="xdsoft_datetimepicker xdsoft_' + options.theme + ' xdsoft_noselect ' + (options.weeks ? ' xdsoft_showweeks' : '') + options.className + '"></div>'),
var datetimepicker = $('<div class="xdsoft_datetimepicker xdsoft_noselect"></div>'),
xdsoft_copyright = $('<div class="xdsoft_copyright"><a target="_blank" href="http://xdsoft.net/jqplugins/datetimepicker/">xdsoft.net</a></div>'),
datepicker = $('<div class="xdsoft_datepicker active"></div>'),
mounth_picker = $('<div class="xdsoft_mounthpicker"><button type="button" class="xdsoft_prev"></button><button type="button" class="xdsoft_today_button"></button>' +
@ -762,6 +762,13 @@
timer1 = 0,
_xdsoft_datetime;
if (options.id) datetimepicker.attr('id', options.id);
if (options.style) datetimepicker.attr('style', options.style);
if (options.weeks) datetimepicker.addClass('xdsoft_showweeks');
datetimepicker.addClass('xdsoft_' + options.theme);
datetimepicker.addClass(options.className);
mounth_picker
.find('.xdsoft_month span')
.after(monthselect);