From 7013e8f5781e838256bf07e7d5de58a974e761a8 Mon Sep 17 00:00:00 2001
From: Evan You <yyx990803@gmail.com>
Date: Thu, 15 Jul 2021 13:45:43 -0400
Subject: [PATCH] fix(runtime-dom): remove class attribute on nullish values

close #3173
---
 packages/runtime-dom/src/modules/class.ts | 25 +++++++++++------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/packages/runtime-dom/src/modules/class.ts b/packages/runtime-dom/src/modules/class.ts
index d26867d9a06..6eecf64169f 100644
--- a/packages/runtime-dom/src/modules/class.ts
+++ b/packages/runtime-dom/src/modules/class.ts
@@ -3,22 +3,21 @@ import { ElementWithTransition } from '../components/Transition'
 // compiler should normalize class + :class bindings on the same element
 // into a single binding ['staticClass', dynamic]
 export function patchClass(el: Element, value: string | null, isSVG: boolean) {
-  if (value == null) {
-    value = ''
+  // directly setting className should be faster than setAttribute in theory
+  // if this is an element during a transition, take the temporary transition
+  // classes into account.
+  const transitionClasses = (el as ElementWithTransition)._vtc
+  if (transitionClasses) {
+    value = (value
+      ? [value, ...transitionClasses]
+      : [...transitionClasses]
+    ).join(' ')
   }
-  if (isSVG) {
+  if (value == null) {
+    el.removeAttribute('class')
+  } else if (isSVG) {
     el.setAttribute('class', value)
   } else {
-    // directly setting className should be faster than setAttribute in theory
-    // if this is an element during a transition, take the temporary transition
-    // classes into account.
-    const transitionClasses = (el as ElementWithTransition)._vtc
-    if (transitionClasses) {
-      value = (value
-        ? [value, ...transitionClasses]
-        : [...transitionClasses]
-      ).join(' ')
-    }
     el.className = value
   }
 }