Merge pull request #2651 from oat-sa/fix/AUT-3890/multiple-hottext-wh…

…en-disallowhtmlinhottext-enabled fix: multiple hottext support when disallowHTMLInHottext is enabled
oat-sa · Dec 20, 2024 · 13c4026 · 13c4026
2 parents 34b9431 + 81c2bf8
commit 13c4026
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/views/js/qtiCreator/widgets/helpers/selectionWrapper.js b/views/js/qtiCreator/widgets/helpers/selectionWrapper.js
@@ -134,10 +134,10 @@ define(['jquery'], function ($) {
              * @param {jQuery} $wrapper - the element that will wrap the selection
              * @returns {boolean}
              */
-            wrapWith: function wrapWith($wrapper) {
-                const range = selection.getRangeAt(0);
+            wrapWith: function wrapWith($wrapper, providedRange) {
+                const range = providedRange || selection.getRangeAt(0);
 
-                if (this.canWrap()) {
+                if (this.canWrap(range)) {
                     try {
                         range.surroundContents($wrapper[0]);
                         selection.removeAllRanges();

diff --git a/views/js/qtiCreator/widgets/interactions/hottextInteraction/states/Question.js b/views/js/qtiCreator/widgets/interactions/hottextInteraction/states/Question.js
@@ -140,7 +140,7 @@ define([
             } else {
                 if (
                     $cloneContent.text() === $cloneContent.html() &&
-                    wrapper.wrapWith($newHottextClone)
+                    wrapper.wrapWith($newHottextClone, range)
                 ) {
                     await this.createNewHottext($newHottextClone);
                 } else {