fix(ssr): fix v-show inline style rendering when style binding is array (vuejs#7814)

jinzhubaofu · yyx990803 · commit 1a979c44d654 · 2018-03-13T11:40:22.000-04:00
fix vuejs#7813
diff --git a/src/platforms/web/server/directives/show.js b/src/platforms/web/server/directives/show.js
@@ -3,6 +3,10 @@
 export default function show (node: VNodeWithData, dir: VNodeDirective) {
   if (!dir.value) {
     const style: any = node.data.style || (node.data.style = {})
-    style.display = 'none'
+    if (Array.isArray(style)) {
+      style.push({ display: 'none' })
+    } else {
+      style.display = 'none'
+    }
   }
 }
diff --git a/test/ssr/ssr-string.spec.js b/test/ssr/ssr-string.spec.js
@@ -267,6 +267,17 @@ describe('SSR: renderToString', () => {
     })
   })
 
+  it('v-show directive merge with style', done => {
+    renderVmWithOptions({
+      template: '<div :style="[{lineHeight: 1}]" v-show="false"><span>inner</span></div>'
+    }, res => {
+      expect(res).toContain(
+        '<div data-server-rendered="true" style="line-height:1;display:none;"><span>inner</span></div>'
+      )
+      done()
+    })
+  })
+
   it('v-show directive not passed to child', done => {
     renderVmWithOptions({
       template: '<foo v-show="false"></foo>',

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,10 @@`
`3`	`3`	`export default function show (node: VNodeWithData, dir: VNodeDirective) {`
`4`	`4`	`if (!dir.value) {`
`5`	`5`	`const style: any = node.data.style \|\| (node.data.style = {})`
`6`		`- style.display = 'none'`
	`6`	`+ if (Array.isArray(style)) {`
	`7`	`+ style.push({ display: 'none' })`
	`8`	`+ } else {`
	`9`	`+ style.display = 'none'`
	`10`	`+ }`
`7`	`11`	`}`
`8`	`12`	`}`