Merge pull request #141 from sumocoders/flatpickr

Fix time pickers
sumocoders · Oct 2, 2024 · d557080 · d557080
2 parents 80f3c22 + 0c3133a
commit d557080
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/src/js/Framework/DateTimePicker/DatePicker.js b/src/js/Framework/DateTimePicker/DatePicker.js
@@ -21,7 +21,7 @@ import { Spanish } from 'flatpickr/dist/l10n/es.js'
 import { Slovenian } from 'flatpickr/dist/l10n/sl.js'
 
 export class DatePicker {
-  constructor (element) {
+  constructor (element, enableTime = false, noCalendar = false) {
     this.element = element
 
     let locale = document.documentElement.lang
@@ -31,7 +31,9 @@ export class DatePicker {
 
     try {
       this.element._flatpickr = flatpickr(this.element, {
-        locale: locale
+        locale: locale,
+        enableTime: enableTime,
+        noCalendar: noCalendar
       })
     } catch (ex) {
       console.log('No translation found for ' + locale)

diff --git a/src/js/Framework/DateTimePicker/DateTimePicker.js b/src/js/Framework/DateTimePicker/DateTimePicker.js
@@ -2,8 +2,6 @@ import { DatePicker } from './DatePicker.js'
 
 export class DateTimePicker extends DatePicker {
   constructor (element) {
-    super(element)
-
-    this.element._flatpickr.config.enableTime = true
+    super(element, true)
   }
 }
diff --git a/src/js/Framework/DateTimePicker/TimePicker.js b/src/js/Framework/DateTimePicker/TimePicker.js
@@ -2,10 +2,7 @@ import { DatePicker } from './DatePicker.js'
 
 export class TimePicker extends DatePicker {
   constructor (element) {
-    super(element)
-
-    this.element._flatpickr.config.enableTime = true
-    this.element._flatpickr.config.noCalendar = true
+    super(element, true, true)
 
     element.parentNode.querySelector('[data-flatpicker-clear]').addEventListener('click', event => this.element._flatpickr.clear())
   }