Skip to content

Commit d7218b6

Browse files
committed
fix(DatePicker, DateRangePicker): add missing name attributes to input elements.
1 parent 52d67bc commit d7218b6

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

js/src/date-range-picker.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@ class DateRangePicker extends Picker {
299299
return date ? (date instanceof Date ? date : new Date(date)) : null
300300
}
301301

302-
_createInput(placeholder, value) {
302+
_createInput(name, placeholder, value) {
303303
const inputEl = document.createElement('input')
304304
inputEl.classList.add('form-control')
305305
inputEl.disabled = this._config.disabled
@@ -308,6 +308,10 @@ class DateRangePicker extends Picker {
308308
inputEl.type = 'text'
309309
inputEl.value = value
310310

311+
if (this._element.id) {
312+
inputEl.name = `${name}-${this._element.id}`
313+
}
314+
311315
return inputEl
312316
}
313317

@@ -319,8 +323,8 @@ class DateRangePicker extends Picker {
319323
inputGroupEl.classList.add(`input-group-${this._config.size}`)
320324
}
321325

322-
const startInputEl = this._createInput(this._getPlaceholder()[0], this._setInputValue(this._startDate))
323-
const endInputEl = this._createInput(this._getPlaceholder()[1], this._setInputValue(this._endDate))
326+
const startInputEl = this._createInput(this._config.range ? 'start-date' : 'date', this._getPlaceholder()[0], this._setInputValue(this._startDate))
327+
const endInputEl = this._createInput('end-date', this._getPlaceholder()[1], this._setInputValue(this._endDate))
324328

325329
const inputGroupTextSeparatorEl = document.createElement('span')
326330
inputGroupTextSeparatorEl.classList.add('input-group-text')

0 commit comments

Comments
 (0)