refactor(plugins): update search plugin

QingWei-Li · QingWei-Li · commit 079bd00395b4 · 2017-02-19T14:38:38.000+08:00
diff --git a/.eslintrc b/.eslintrc
@@ -2,5 +2,9 @@
   "extends": ["vue"],
   "env": {
     "browser": true
+  },
+  "globals": {
+    "Docsify": true,
+    "$docsify": true
   }
 }
diff --git a/build/build.js b/build/build.js
@@ -32,26 +32,26 @@ build({
   plugins: [commonjs(), nodeResolve()]
 })
 
-// build({
-//   entry: 'plugins/search.js',
-//   output: 'plugins/search.js',
-//   moduleName: 'D.Search'
-// })
+build({
+  entry: 'plugins/search/index.js',
+  output: 'plugins/search.js',
+  moduleName: 'D.Search'
+})
 
-// build({
-//   entry: 'plugins/ga.js',
-//   output: 'plugins/ga.js',
-//   moduleName: 'D.GA'
-// })
+build({
+  entry: 'plugins/ga.js',
+  output: 'plugins/ga.js',
+  moduleName: 'D.GA'
+})
 
 if (isProd) {
   build({
-    entry: 'index.js',
+    entry: 'core/index.js',
     output: 'docsify.min.js',
     plugins: [commonjs(), nodeResolve(), uglify()]
   })
   build({
-    entry: 'plugins/search.js',
+    entry: 'plugins/search/index.js',
     output: 'plugins/search.min.js',
     moduleName: 'D.Search',
     plugins: [uglify()]
diff --git a/docs/index.html b/docs/index.html
@@ -10,7 +10,7 @@
   <link rel="stylesheet" href="//unpkg.com/docsify/lib/themes/vue.css">
 </head>
 <body>
-  <nav>
+  <nav data-cloak>
     <a href="#/">En</a>
     <a href="#/zh-cn/">中文</a>
   </nav>
diff --git a/docs/plugins.md b/docs/plugins.md
@@ -90,7 +90,7 @@ window.$docsify = {
     })
 
     hook.ready(function() {
-      // Called after initialization is complete. Only trigger once, no arguments.
+      // Called after initial completion, no arguments.
     })
   }
  ]
diff --git a/docs/zh-cn/plugins.md b/docs/zh-cn/plugins.md
@@ -85,7 +85,7 @@ window.$docsify = {
     })
 
     hook.ready(function() {
-      // 初始化完成后调用，只调用一次，没有参数。
+      // 初始化并第一次加完成数据后调用，没有参数。
     })
   }
  ]
diff --git a/package.json b/package.json
@@ -12,7 +12,7 @@
     "build": "rm -rf lib themes && node build/build.js && mkdir lib/themes && mkdir themes && node build/build-css.js",
     "dev:build": "rm -rf lib themes && mkdir themes && node build/build.js --dev && node build/build-css.js --dev",
     "dev": "node app.js & nodemon -w src -e js,css --exec 'npm run dev:build'",
-    "test": "eslint src test"
+    "test": "eslint src"
   },
   "repository": {
     "type": "git",
diff --git a/src/core/config.js b/src/core/config.js
@@ -39,4 +39,6 @@ if (script) {
   if (config.name === true) config.name = ''
 }
 
+window.$docsify = config
+
 export default config
diff --git a/src/core/event/scroll.js b/src/core/event/scroll.js
@@ -45,8 +45,8 @@ export function scrollActiveSidebar () {
     const li = nav[last.getAttribute('data-id')]
 
     if (!li || li === active) return
-    if (active) active.classList.remove('active')
 
+    active && active.classList.remove('active')
     li.classList.add('active')
     active = li
 
@@ -79,7 +79,7 @@ export function scrollActiveSidebar () {
 
 export function scrollIntoView (id) {
   const section = dom.find('#' + id)
-  section && setTimeout(() => section.scrollIntoView(), 0)
+  section && section.scrollIntoView()
 }
 
 const scrollEl = dom.$.scrollingElement || dom.$.documentElement
diff --git a/src/core/fetch/index.js b/src/core/fetch/index.js
@@ -57,15 +57,16 @@ export function fetchMixin (proto) {
       .then(text => this._renderCover(text))
   }
 
-  proto.$fetch = function () {
+  proto.$fetch = function (cb = noop) {
     this._fetchCover()
     this._fetch(result => {
       this.$resetEvents()
       callHook(this, 'doneEach')
+      cb()
     })
   }
 }
 
 export function initFetch (vm) {
-  vm.$fetch()
+  vm.$fetch(_ => callHook(vm, 'ready'))
 }
diff --git a/src/core/global-api.js b/src/core/global-api.js
@@ -1,7 +1,7 @@
 import * as util from './util'
 import * as dom from './util/dom'
 import * as render from './render/compiler'
-import * as route from './route/util'
+import * as route from './route/hash'
 import { get } from './fetch/ajax'
 import marked from 'marked'
 import prism from 'prismjs'
diff --git a/src/core/index.js b/src/core/index.js
@@ -25,4 +25,5 @@ initGlobalAPI()
 /**
  * Run Docsify
  */
-new Docsify()
+
+setTimeout(_ => new Docsify(), 0)
diff --git a/src/core/init/index.js b/src/core/init/index.js
@@ -18,7 +18,6 @@ export function initMixin (proto) {
     initEvent(vm) // Bind events
     initRoute(vm) // Add hashchange eventListener
     initFetch(vm) // Fetch data
-    callHook(vm, 'ready')
   }
 }
 
diff --git a/src/core/render/compiler.js b/src/core/render/compiler.js
@@ -5,7 +5,7 @@ import { genTree } from './gen-tree'
 import { slugify, clearSlugCache } from './slugify'
 import { emojify } from './emojify'
 import { toURL, parse } from '../route/hash'
-import { getBasePath, isResolvePath, getPath } from '../route/util'
+import { getBasePath, isAbsolutePath, getPath } from '../route/util'
 import { isFn, merge, cached } from '../util/core'
 
 let markdownCompiler = marked
@@ -87,7 +87,7 @@ renderer.image = function (href, title, text) {
   let url = href
   const titleHTML = title ? ` title="${title}"` : ''
 
-  if (!isResolvePath(href)) {
+  if (!isAbsolutePath(href)) {
     url = getPath(contentBase, href)
   }
 
diff --git a/src/core/render/index.js b/src/core/render/index.js
@@ -5,7 +5,7 @@ import cssVars from '../util/polyfill/css-vars'
 import * as tpl from './tpl'
 import { markdown, sidebar, subSidebar, cover } from './compiler'
 import { callHook } from '../init/lifecycle'
-import { getBasePath, getPath, isResolvePath } from '../route/util'
+import { getBasePath, getPath, isAbsolutePath } from '../route/util'
 
 function executeScript () {
   const script = dom.findAll('.markdown-section>script')
@@ -101,7 +101,7 @@ export function renderMixin (proto) {
         let path = m[1]
 
         dom.toggleClass(el, 'add', 'has-mask')
-        if (isResolvePath(m[1])) {
+        if (isAbsolutePath(m[1])) {
           path = getPath(getBasePath(this.config.basePath), m[1])
         }
         el.style.backgroundImage = `url(${path})`
@@ -157,4 +157,5 @@ export function initRender (vm) {
     // Polyfll
     cssVars(config.themeColor)
   }
+  dom.toggleClass(dom.body, 'ready')
 }
diff --git a/src/core/route/hash.js b/src/core/route/hash.js
@@ -1,5 +1,6 @@
 import { merge, cached } from '../util/core'
 import { parseQuery, stringifyQuery, cleanPath } from './util'
+export * from './util'
 
 function replaceHash (path) {
   const i = window.location.href.indexOf('#')
diff --git a/src/core/route/index.js b/src/core/route/index.js
@@ -1,10 +1,9 @@
 import { normalize, parse } from './hash'
-import { getBasePath, getPath } from './util'
+import { getBasePath, getPath, isAbsolutePath } from './util'
 import { on } from '../util/dom'
 
 function getAlias (path, alias) {
-  if (alias[path]) return getAlias(alias[path], alias)
-  return path
+  return alias[path] ? getAlias(alias[path], alias) : path
 }
 
 function getFileName (path) {
@@ -24,7 +23,7 @@ export function routeMixin (proto) {
     path = getAlias(path, config.alias)
     path = getFileName(path)
     path = path === '/README.md' ? (config.homepage || path) : path
-    path = getPath(base, path)
+    path = isAbsolutePath(path) ? path : getPath(base, path)
 
     return path
   }
diff --git a/src/core/route/util.js b/src/core/route/util.js
@@ -25,7 +25,7 @@ export function stringifyQuery (obj) {
   const qs = []
 
   for (const key in obj) {
-    qs.push(`${encode(key)}=${encode(obj[key])}`)
+    qs.push(`${encode(key)}=${encode(obj[key])}`.toLowerCase())
   }
 
   return qs.length ? `?${qs.join('&')}` : ''
@@ -41,8 +41,8 @@ export function getPath (...args) {
   return cleanPath(args.join('/'))
 }
 
-export const isResolvePath = cached(path => {
-  return /:|(\/{2})/.test(path)
+export const isAbsolutePath = cached(path => {
+  return /(:|(\/{2}))/.test(path)
 })
 
 export const getRoot = cached(path => {
diff --git a/src/plugins/ga.js b/src/plugins/ga.js
@@ -1,5 +1,4 @@
 // From https://github.com/egoist/vue-ga/blob/master/src/index.js
-
 function appendScript () {
   const script = document.createElement('script')
   script.async = true
@@ -24,19 +23,13 @@ function collect () {
   window.ga('send', 'pageview')
 }
 
-const install = function () {
-  if (install.installed) return
-  install.installed = true
-
+const install = function (hook) {
   if (!window.$docsify.ga) {
     console.error('[Docsify] ga is required.')
     return
   }
 
-  window.$docsify.plugins = [].concat(function (hook) {
-    hook.init(collect)
-    hook.beforeEach(collect)
-  }, window.$docsify.plugins)
+  hook.beforeEach(collect)
 }
 
-export default install()
+window.$docsify.plugins = [].concat(install, window.$docsify.plugins)
diff --git a/src/plugins/search.js b/src/plugins/search.js
diff --git a/src/plugins/search/component.js b/src/plugins/search/component.js
diff --git a/src/plugins/search/index.js b/src/plugins/search/index.js
diff --git a/src/plugins/search/search.js b/src/plugins/search/search.js

Original file line number	Diff line number	Diff line change
`@@ -2,5 +2,9 @@`
`2`	`2`	`"extends": ["vue"],`
`3`	`3`	`"env": {`
`4`	`4`	`"browser": true`
	`5`	`+ },`
	`6`	`+ "globals": {`
	`7`	`+ "Docsify": true,`
	`8`	`+ "$docsify": true`
`5`	`9`	`}`
`6`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ window.$docsify = {`
`90`	`90`	`})`
`91`	`91`
`92`	`92`	`hook.ready(function() {`
`93`		`- // Called after initialization is complete. Only trigger once, no arguments.`
	`93`	`+ // Called after initial completion, no arguments.`
`94`	`94`	`})`
`95`	`95`	`}`
`96`	`96`	`]`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ window.$docsify = {`
`85`	`85`	`})`
`86`	`86`
`87`	`87`	`hook.ready(function() {`
`88`		`- // 初始化完成后调用，只调用一次，没有参数。`
	`88`	`+ // 初始化并第一次加完成数据后调用，没有参数。`
`89`	`89`	`})`
`90`	`90`	`}`
`91`	`91`	`]`
Original file line number	Diff line number	Diff line change
`@@ -39,4 +39,6 @@ if (script) {`
`39`	`39`	`if (config.name === true) config.name = ''`
`40`	`40`	`}`
`41`	`41`
	`42`	`+window.$docsify = config`
	`43`	`+`
`42`	`44`	`export default config`
Original file line number	Diff line number	Diff line change
`@@ -57,15 +57,16 @@ export function fetchMixin (proto) {`
`57`	`57`	`.then(text => this._renderCover(text))`
`58`	`58`	`}`
`59`	`59`
`60`		`- proto.$fetch = function () {`
	`60`	`+ proto.$fetch = function (cb = noop) {`
`61`	`61`	`this._fetchCover()`
`62`	`62`	`this._fetch(result => {`
`63`	`63`	`this.$resetEvents()`
`64`	`64`	`callHook(this, 'doneEach')`
	`65`	`+ cb()`
`65`	`66`	`})`
`66`	`67`	`}`
`67`	`68`	`}`
`68`	`69`
`69`	`70`	`export function initFetch (vm) {`
`70`		`- vm.$fetch()`
	`71`	`+ vm.$fetch(_ => callHook(vm, 'ready'))`
`71`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,6 @@ export function initMixin (proto) {`
`18`	`18`	`initEvent(vm) // Bind events`
`19`	`19`	`initRoute(vm) // Add hashchange eventListener`
`20`	`20`	`initFetch(vm) // Fetch data`
`21`		`- callHook(vm, 'ready')`
`22`	`21`	`}`
`23`	`22`	`}`
`24`	`23`