fix(tooltip): don't sanitize empty text, fixes empty tooltip being sh…

…own (#1495)
ghiscoding · May 1, 2024 · dcc693b · dcc693b
1 parent 4c63bcd
commit dcc693b
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/packages/common/src/services/domUtilities.ts b/packages/common/src/services/domUtilities.ts
@@ -127,10 +127,12 @@ export function buildMsSelectCollectionList(type: 'editor' | 'filter', collectio
  */
 export function sanitizeTextByAvailableSanitizer(gridOptions: GridOption, dirtyHtml: string, sanitizerOptions?: DOMPurify.Config): string {
   let sanitizedText = dirtyHtml;
-  if (typeof gridOptions?.sanitizer === 'function') {
-    sanitizedText = gridOptions.sanitizer(dirtyHtml || '');
-  } else if (typeof DOMPurify?.sanitize === 'function') {
-    sanitizedText = (DOMPurify.sanitize(dirtyHtml || '', sanitizerOptions || { ADD_ATTR: ['level'], RETURN_TRUSTED_TYPE: true }) || '') as string;
+  if (dirtyHtml) {
+    if (typeof gridOptions?.sanitizer === 'function') {
+      sanitizedText = gridOptions.sanitizer(dirtyHtml || '');
+    } else if (typeof DOMPurify?.sanitize === 'function') {
+      sanitizedText = (DOMPurify.sanitize(dirtyHtml || '', sanitizerOptions || { ADD_ATTR: ['level'], RETURN_TRUSTED_TYPE: true }) || '') as string;
+    }
   }
 
   return sanitizedText;