From 2525091f8704616732ac524b416de44cd79cc8fd Mon Sep 17 00:00:00 2001 From: David Karlsson Date: Thu, 2 Nov 2023 16:00:36 +0100 Subject: [PATCH] Check if addClass is truthy before adding it to element DOMTokenList::add() throws a DOMException if you try to add an empty string as a class. This check prevents this from happening so that you can set ValidationInputValidCssClassName etc. to an empty string if you don't want to add classes for valid input fields or messages for instance. --- src/index.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/index.ts b/src/index.ts index f45b29a..b5698b3 100644 --- a/src/index.ts +++ b/src/index.ts @@ -1254,7 +1254,7 @@ export class ValidationService { * @param removeClass Class to remove */ private swapClasses(element: Element, addClass: string, removeClass: string) { - if (!element.classList.contains(addClass)) { + if (addClass && !element.classList.contains(addClass)) { element.classList.add(addClass); } if (element.classList.contains(removeClass)) {