[core] Comment on CSS.escape for the future

oliviertassinari · oliviertassinari · commit ed3472b0e2b3 · 2024-05-31T14:23:50.000+02:00
diff --git a/packages/x-data-grid/src/utils/domUtils.ts b/packages/x-data-grid/src/utils/domUtils.ts
@@ -10,6 +10,8 @@ export function findParentElementFromClassName(elem: Element, className: string)
   return elem.closest(`.${className}`);
 }
 
+// TODO, eventually replaces this function with CSS.escape, once available in jsdom, either added manually or built in
+// https://github.com/jsdom/jsdom/issues/1550#issuecomment-236734471
 export function escapeOperandAttributeSelector(operand: string): string {
   return operand.replace(/["\\]/g, '\\$&');
 }

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,8 @@ export function findParentElementFromClassName(elem: Element, className: string)`
`10`	`10`	return elem.closest(`.${className}`);
`11`	`11`	`}`
`12`	`12`
	`13`	`+// TODO, eventually replaces this function with CSS.escape, once available in jsdom, either added manually or built in`
	`14`	`+// https://github.com/jsdom/jsdom/issues/1550#issuecomment-236734471`
`13`	`15`	`export function escapeOperandAttributeSelector(operand: string): string {`
`14`	`16`	`return operand.replace(/["\\]/g, '\\$&');`
`15`	`17`	`}`