From 47c7b2f4eee1cf018701ef0b061c464157086ca2 Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <1697880+AngelFQC@users.noreply.github.com>
Date: Fri, 31 Jan 2025 16:17:20 -0500
Subject: [PATCH] Display: Fix template for DatePicker element

---
 .../lib/formvalidator/Element/DatePicker.php  | 27 ++++++++-----------
 src/CoreBundle/Component/Utils/ActionIcon.php |  2 ++
 2 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/public/main/inc/lib/formvalidator/Element/DatePicker.php b/public/main/inc/lib/formvalidator/Element/DatePicker.php
index eb09e3f278..a92af87f39 100644
--- a/public/main/inc/lib/formvalidator/Element/DatePicker.php
+++ b/public/main/inc/lib/formvalidator/Element/DatePicker.php
@@ -2,7 +2,8 @@
 
 /* For licensing terms, see /license.txt */
 
-use Chamilo\CoreBundle\Component\Utils\ToolIcon;
+use Chamilo\CoreBundle\Component\Utils\ActionIcon;
+use Chamilo\CoreBundle\Component\Utils\ObjectIcon;
 use Chamilo\CoreBundle\Framework\Container;
 
 /**
@@ -20,7 +21,7 @@ class DatePicker extends HTML_QuickForm_text
         if (!isset($attributes['id'])) {
             $attributes['id'] = $elementName;
         }
-        $attributes['class'] = 'form-control border flex-grow';
+        $attributes['class'] = 'p-component p-inputtext p-filled';
 
         parent::__construct($elementName, $elementLabel, $attributes);
         $this->_appendName = true;
@@ -57,20 +58,18 @@ class DatePicker extends HTML_QuickForm_text
             $requiredSymbol = '<span class="form_required">*</span>';
         }
 
-        $attrs = $this->_attributes;
-        unset($attrs['id']);
+        $this->setAttribute('placeholder', get_lang('Select date'));
 
         return '
-        <div>'.$requiredSymbol.$label.'</div>
-        <div id="'.$id.'" class="flex items-center mt-1 flatpickr-wrapper" data-wrap="true">
-            <input '.$this->_getAttrString($this->_attributes).'
-                value="'.$value.'" placeholder="'.get_lang('Select date').'" data-input>
+        <label>'.$requiredSymbol.$label.'</label>
+        <div id="'.$id.'_container" class="flex items-center mt-1 flatpickr-wrapper" data-wrap="true">
+            <input '.$this->_getAttrString($this->_attributes).' value="'.$value.'" data-input>
             <div class="flex space-x-1 ml-2" id="button-addon3">
                 <button class="btn btn--secondary-outline mr-2" type="button" data-toggle>
-                    <i class="pi pi-calendar pi-lg"></i>
+                  '.Display::getMdiIcon(ObjectIcon::AGENDA).'
                 </button>
                 <button class="btn btn--secondary-outline" type="button" data-clear>
-                    <i class="pi pi-times pi-lg"></i>
+                  '.Display::getMdiIcon(ActionIcon::CLOSE).'
                 </button>
             </div>
         </div>
@@ -109,7 +108,7 @@ class DatePicker extends HTML_QuickForm_text
         return "<script>
         document.addEventListener('DOMContentLoaded', function () {
             function initializeFlatpickr() {
-                const fp = flatpickr('#{$id}', {
+                const fp = flatpickr('#{$id}_container', {
                     locale: '{$localeCode}',
                     altInput: true,
                     altFormat: '{$altFormat}',
@@ -119,11 +118,7 @@ class DatePicker extends HTML_QuickForm_text
                     wrap: true
                 });
 
-                if ($('label[for=\"".$id."\"]').length > 0) {
-                    $('label[for=\"".$id."\"]').hide();
-                }
-
-                document.querySelector('label[for=\"' + '{$id}' + '\"]').classList.add('datepicker-label');
+                $('label[for=\"".$id."\"]').hide().addClass('datepicker-label');
             }
 
             function loadLocale() {
diff --git a/src/CoreBundle/Component/Utils/ActionIcon.php b/src/CoreBundle/Component/Utils/ActionIcon.php
index 2c9aa0ba57..0eabb681b9 100644
--- a/src/CoreBundle/Component/Utils/ActionIcon.php
+++ b/src/CoreBundle/Component/Utils/ActionIcon.php
@@ -164,4 +164,6 @@ enum ActionIcon: string
     case SWAP_FILE = 'file-swap';
 
     case ADD_FILE_VARIATION = 'file-replace';
+
+    case CLOSE = 'close';
 }