feat: Remove "manual" importStrategy - makes no sense nowadays since auto-import is so powerful

Author: rstoenescu

generator/index.js

@@ -23,23 +23,6 @@ const iconMap = {
   mdi: 'mdi-v4'
 }
 
-const components = [
-  'QLayout',
-  'QHeader',
-  'QDrawer',
-  'QPageContainer',
-  'QPage',
-  'QToolbar',
-  'QToolbarTitle',
-  'QBtn',
-  'QIcon',
-  'QList',
-  'QItem',
-  'QItemSection',
-  'QItemLabel'
-]
-
-const directives = []
 const plugins = []
 
 module.exports = (api, opts) => {
@@ -83,7 +66,7 @@ module.exports = (api, opts) => {
     pluginOptions.quasar = Object.assign(
       pluginOptions.quasar || {},
       {
-        importStrategy: opts.quasar.importStrategy,
+        importStrategy: 'kebab',
         rtlSupport: opts.quasar.rtlSupport
       }
     )
@@ -122,7 +105,6 @@ module.exports = (api, opts) => {
     let lines = `import Vue from 'vue'\n`
 
     const
-      autoImport = opts.quasar.importStrategy !== 'manual',
       hasIconSet = opts.quasar.iconSet !== 'material-icons',
       hasLang = opts.quasar.lang !== 'en-us'
 
@@ -158,47 +140,12 @@ module.exports = (api, opts) => {
       })
 
     // build import
-    if (autoImport) {
-      lines += `\nimport { Quasar } from 'quasar'`
-    }
-    else {
-      lines += `\nimport {\n  Quasar, `
-      components
-        .concat(directives)
-        .concat(plugins)
-        .forEach(part => {
-          lines += `\n  ${part},`
-        })
-      lines += `\n}`
-      lines += ` from 'quasar'`
-    }
+    lines += `\nimport { Quasar } from 'quasar'`
 
     // build Vue.use()
     lines += `\n\nVue.use(Quasar, {`
     lines += `\n  config: {}`
 
-    lines += ',\n  components: {'
-    if (autoImport) {
-      lines += ` /* not needed if importStrategy is not 'manual' */ }`
-    }
-    else {
-      components.forEach(part => {
-        lines += `\n    ${part},`
-      })
-      lines += `\n  }`
-    }
-
-    lines += ',\n  directives: {'
-    if (autoImport) {
-      lines += ` /* not needed if importStrategy is not 'manual' */ }`
-    }
-    else {
-      directives.forEach(part => {
-        lines += `\n   ${part},`
-      })
-      lines += `\n  }`
-    }
-
     lines += ',\n  plugins: {'
     plugins.forEach(part => {
       lines += `\n   ${part},`

index.js

@@ -38,7 +38,7 @@ module.exports = (api, options) => {
 
     chain.performance.maxEntrypointSize(512000)
 
-    const strategy = options.pluginOptions.quasar.importStrategy
+    const strategy = options.pluginOptions.quasar.importStrategy || 'kebab'
 
     if (['kebab', 'pascal', 'combined'].includes(strategy)) {
       chain.module.rule('vue')
@@ -47,9 +47,9 @@ module.exports = (api, options) => {
         .options(strategy)
         .before('cache-loader')
     }
-    else if (strategy !== void 0 && strategy !== 'manual') {
+    else {
       console.error(`Incorrect setting for quasar > importStrategy (${strategy})`)
-      console.error(`Use one of: 'kebab', 'pascal', 'combined', 'manual'.`)
+      console.error(`Use one of: 'kebab', 'pascal', 'combined'.`)
       console.log()
       process.exit(1)
     }

prompts.js

@@ -8,25 +8,6 @@ module.exports = [
     default: true
   },
 
-  {
-    name: 'quasar.importStrategy',
-    type: 'list',
-    message: 'Pick a Quasar components & directives import strategy: (can be changed later)',
-    choices: [
-      {
-        name: `Auto-import in-use Quasar components & directives (kebab-case, can be later changed to 'pascal'/'combined'/'manual')`,
-        value: 'kebab',
-        short: 'Auto import (kebab-case)',
-        checked: true
-      },
-      {
-        name: 'Manually specify what to import',
-        value: 'manual',
-        short: 'Manual'
-      }
-    ]
-  },
-
   {
     name: 'quasar.cssPreprocessor',
     type: 'list',