Skip to content

Vue left menu v1

Eric Eastwood requested to merge 1987-vue-left-menu into develop

Vue left menu v1 (part of https://gitlab.com/gitlab-org/gitter/webapp/issues/1987)

  • All of this is behind a feature toggle
  • No CSS transitions
  • Only works on room/chat page and userhome(/home).
    • The old left menu will show on other pages but is broken
  • Search: only messages
  • Mobile and NLI(not logged in) are not considered
  • No favourite room drag and dropping to order
  • No groups/community view

Sub-MRs

Get it running locally

The Vue left menu is used on the chat and userhome view for now

Notes


Testing notes

See https://gitlab.com/gitlab-org/gitter/webapp/merge_requests/1404#testing-notes


Error: ReferenceError: window is not defined
    at server-bundle.js:309:2
    at server-bundle.js:298:46
    at module.exports../node_modules/style-loader/lib/addStyles.js.module.exports (node_modules/style-loader/lib/addStyles.js:80:0)
    at Object../node_modules/style-loader/index.js?!./node_modules/css-loader/dist/cjs.js?!./node_modules/vue-loader/lib/loaders/stylePostLoader.js!./node_modules/postcss-loader/src/index.js?!./node_modules/vue-loader/lib/index.js?!./public/js/vue/left-menu/components/index.vue?vue&type=style&index=0&id=44ebcb1a&scoped=true&lang=css& (public/js/vue/left-menu/components/index.vue?300e:16:81)
    at __webpack_require__ (webpack/bootstrap:19:0)
    at Module../public/js/vue/left-menu/components/index.vue?vue&type=style&index=0&id=44ebcb1a&scoped=true&lang=css& (public/js/vue/left-menu/components/index.vue?8f3b:1:0)
    at __webpack_require__ (webpack/bootstrap:19:0)
    at VueComponent.injectStyles (public/js/vue/left-menu/components/index.vue:6:15)
    at VueComponent.hook (node_modules/vue-loader/lib/runtime/componentNormalizer.js:53:0)
    at invokeWithErrorHandling (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue\dist\vue.runtime.common.dev.js:1850:57)
    at callHook (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue\dist\vue.runtime.common.dev.js:4182:7)
    at VueComponent.Vue._init (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue\dist\vue.runtime.common.dev.js:4950:5)
    at new VueComponent (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue\dist\vue.runtime.common.dev.js:5099:12)
    at createComponentInstanceForVnode (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8140:10)
    at renderComponentInner (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8364:40)
    at renderComponent (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8339:5)
    at renderNode (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8250:5)
    at resolve (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8612:7)
    at waitForServerPrefetch (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8243:3)
    at render (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:8614:5)
    at Object.renderToString (C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:9139:9)
    at C:\Users\MLM\Documents\GitLab\webapp\node_modules\vue-server-renderer\build.dev.js:9505:22
    at process._tickCallback (internal/process/next_tick.js:68:7)

Todo



Part of https://gitlab.com/gitlab-org/gitter/webapp/issues/1987

Closes https://gitlab.com/gitlab-org/gitter/webapp/issues/2096

Edited by Eric Eastwood

Merge request reports