MDL-75669 theme_boost: Fix Bootstrap related JS

roland04 · Nov 29, 2024 · cb6be31 · cb6be31
1 parent 1f50445
commit cb6be31
Show file tree

Hide file tree

Showing 21 changed files with 92 additions and 113 deletions.
diff --git a/calendar/amd/build/popover.min.js b/calendar/amd/build/popover.min.js
diff --git a/calendar/amd/build/popover.min.js.map b/calendar/amd/build/popover.min.js.map
diff --git a/calendar/amd/src/popover.js b/calendar/amd/src/popover.js
@@ -22,7 +22,7 @@
  * @since 4.0
  */
 
-import 'theme_boost/popover';
+import Popover from 'theme_boost/bootstrap/popover';
 import jQuery from 'jquery';
 import * as CalendarSelectors from 'core_calendar/selectors';
 
@@ -36,12 +36,11 @@ const isPopoverAvailable = (dateContainer) => {
 };
 
 const isPopoverConfigured = new Map();
-
+// TODO: Refactor this to use the new Popover class.
 const showPopover = target => {
     const dateContainer = target.closest(CalendarSelectors.elements.dateContainer);
     if (!isPopoverConfigured.has(dateContainer)) {
-        const dateEle = jQuery(target);
-        dateEle.popover({
+        const config = {
             trigger: 'manual',
             placement: 'top',
             html: true,
@@ -56,7 +55,8 @@ const showPopover = target => {
                 return content.html();
             },
             'animation': false,
-        });
+        };
+        new Popover(target, config);
 
         isPopoverConfigured.set(dateContainer, true);
     }