fix(style): always set new styles

yyx990803 · yyx990803 · commit f5ef882a781b · 2023-12-06T23:17:02.000+08:00
close #12901 close #12946
diff --git a/src/platforms/web/runtime/modules/style.ts b/src/platforms/web/runtime/modules/style.ts
@@ -91,10 +91,8 @@ function updateStyle(oldVnode: VNodeWithData, vnode: VNodeWithData) {
   }
   for (name in newStyle) {
     cur = newStyle[name]
-    if (cur !== oldStyle[name]) {
-      // ie9 setting to null has no effect, must use empty string
-      setProp(el, name, cur == null ? '' : cur)
-    }
+    // ie9 setting to null has no effect, must use empty string
+    setProp(el, name, cur == null ? '' : cur)
   }
 }
 
diff --git a/test/unit/modules/vdom/modules/style.spec.ts b/test/unit/modules/vdom/modules/style.spec.ts
@@ -36,4 +36,19 @@ describe('vdom style module', () => {
     expect(elm.style.fontSize).toBe('')
     expect(elm.style.display).toBe('block')
   })
+
+  it('border related style should update correctly', () => {
+    const vnode1 = new VNode('p', {
+      style: { border: '10px solid red', 'border-bottom': '10px solid blue' }
+    })
+    const vnode2 = new VNode('p', {
+      style: {
+        'border-right': '10px solid red',
+        'border-bottom': '10px solid blue'
+      }
+    })
+    patch(null, vnode1)
+    const elm = patch(vnode1, vnode2)
+    expect(elm.style.borderBottom).toBe('10px solid blue')
+  })
 })

Original file line number	Diff line number	Diff line change
`@@ -91,10 +91,8 @@ function updateStyle(oldVnode: VNodeWithData, vnode: VNodeWithData) {`
`91`	`91`	`}`
`92`	`92`	`for (name in newStyle) {`
`93`	`93`	`cur = newStyle[name]`
`94`		`- if (cur !== oldStyle[name]) {`
`95`		`- // ie9 setting to null has no effect, must use empty string`
`96`		`- setProp(el, name, cur == null ? '' : cur)`
`97`		`- }`
	`94`	`+ // ie9 setting to null has no effect, must use empty string`
	`95`	`+ setProp(el, name, cur == null ? '' : cur)`
`98`	`96`	`}`
`99`	`97`	`}`
`100`	`98`