Merge pull request #31859 from colemanw/iconPickerFix

Improve crmUiIconPicker with ngModel support for binding
civicrm · Jan 27, 2025 · 24b66b4 · 24b66b4
2 parents 1c9cc60 + f0e40a5
commit 24b66b4
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/ang/crmUi.js b/ang/crmUi.js
@@ -1294,10 +1294,21 @@
     .directive('crmUiIconPicker', function($timeout) {
       return {
         restrict: 'A',
-        controller: function($element) {
+        require: '?ngModel', // Soft require ngModel
+        controller: function($element, $scope, $attrs) {
           CRM.loadScript(CRM.config.resourceBase + 'js/jquery/jquery.crmIconPicker.js').then(function() {
             $timeout(function() {
               $element.crmIconPicker();
+
+              // If ngModel is present, set up two-way binding
+              if ($attrs.ngModel) {
+                $scope.$watch($attrs.ngModel, function(newValue) {
+                  if (newValue !== undefined) {
+                    // Update the value in the picker
+                    $element.val(newValue).trigger('change');
+                  }
+                });
+              }
             });
           });
         }