Issue #3513683 by ahmad abbad: Required Validation Issue with ACE Editor Field

Remaining tasks

• File an issue
• Addition/Change/Update/Fix
• Testing to ensure no regression
• Automated unit testing coverage
• Automated functional testing coverage
• UX/UI designer responsibilities
• Readability
• Accessibility
• Performance
• Security
• Documentation
• Code review by maintainers
• Full testing and approval
• Credit contributors
• Review with the product owner
• Release notes snippet
• Release

Closes #3513683

js/editor.js

@@ -2,6 +2,8 @@
     'use strict';
     // A page may contain multiple editors. editors variable store all of them as { id: editor_object }
     var editors = {};
+    // Store required elements to restore them later
+    var requiredElements = new Set();
     /**
      * @file
      * Defines AceEditor as a Drupal editor.
@@ -18,7 +20,13 @@
 
             // Creating a unique id for our new text editor
             var ace_editor_id = element_id+"-ace-editor";
-
+            // Handle the 'required' attribute
+            if (element.hasAttribute('required')) {
+              // Store the ID of the required element
+              requiredElements.add(element_id);
+              // Remove the 'required' attribute
+              element.removeAttribute('required');
+            }
             // We don't delete the original textarea, but hide it.
             $element.hide().css('visibility', 'hidden');
 
@@ -64,7 +72,11 @@
             else{
                 editors[ace_editor_id].destroy();
                 editors[ace_editor_id].container.remove();
-
+                // Restore the 'required' attribute if it was originally present
+                if (requiredElements.has(element_id)) {
+                  element.setAttribute('required', 'required');
+                  requiredElements.delete(element_id); // Remove from the set
+                }
                 $element.show().css('visibility', 'visible');
                 //element.removeAttribute('contentEditable');
             }