fix: dont change title when value is undefined (fix #396)

nuxt · Jun 22, 2019 · 90f9710 · 90f9710
1 parent 4ca3050
commit 90f9710
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 10 deletions.
diff --git a/examples/basic/index.html b/examples/basic/index.html
@@ -1,5 +1,12 @@
 <!DOCTYPE html>
-<link rel="stylesheet" href="/global.css">
-<a href="/">&larr; Examples index</a>
-<div id="app"></div>
-<script src="/__build__/basic.js"></script>
+<html>
+<head>
+  <title>Basic Title</title>
+  <link rel="stylesheet" href="/global.css">
+</head>
+<body>
+  <a href="/">&larr; Examples index</a>
+  <div id="app"></div>
+  <script src="/__build__/basic.js"></script>
+</body>
+</html>
diff --git a/src/client/updaters/title.js b/src/client/updaters/title.js
@@ -3,6 +3,10 @@
  *
  * @param  {String} title - the new title of the document
  */
-export default function updateTitle(title = document.title) {
+export default function updateTitle(title) {
+  if (title === undefined) {
+    return
+  }
+
   document.title = title
 }
diff --git a/src/shared/constants.js b/src/shared/constants.js
@@ -4,7 +4,7 @@
 
 // set some sane defaults
 export const defaultInfo = {
-  title: '',
+  title: undefined,
   titleChunk: '',
   titleTemplate: '%s',
   htmlAttrs: {},

diff --git a/test/unit/components.test.js b/test/unit/components.test.js
@@ -90,7 +90,7 @@ describe('client', () => {
 
     jest.runAllTimers()
     metaInfo = getMetaInfo(wrapper.vm.$parent)
-    expect(metaInfo.title).toEqual('')
+    expect(metaInfo.title).toEqual(undefined)
   })
 
   test('meta-info can be rendered with inject', () => {

diff --git a/test/unit/getMetaInfo.test.js b/test/unit/getMetaInfo.test.js
@@ -13,7 +13,7 @@ describe('getMetaInfo', () => {
     const component = new Vue()
 
     expect(getMetaInfo(component)).toEqual({
-      title: '',
+      title: undefined,
       titleChunk: '',
       titleTemplate: '%s',
       htmlAttrs: {},
@@ -654,7 +654,7 @@ describe('getMetaInfo', () => {
     })
 
     expect(getMetaInfo(component)).toEqual({
-      title: '',
+      title: undefined,
       titleChunk: '',
       titleTemplate: '%s',
       htmlAttrs: {},
@@ -802,7 +802,7 @@ describe('getMetaInfo', () => {
     })
 
     expect(getMetaInfo(component)).toEqual({
-      title: '',
+      title: undefined,
       titleChunk: '',
       titleTemplate: '%s',
       htmlAttrs: {},