Merge pull request #55 from ojvribeiro/fix-favicon-in-app

fix(core): make use of favicon more flexible

.vulmix/mix.js

@@ -1,5 +1,6 @@
 const mix = require('laravel-mix')
 const path = require('path')
+const fs = require('fs')
 
 require('laravel-mix-serve')
 require('laravel-mix-simple-image-processing')
@@ -40,8 +41,6 @@ class VulmixInit {
         }
       )
 
-      .copy('.vulmix/assets/icons/favicon-16x16.png', '_dist/assets/icons')
-
       .sass(
         '.vulmix/assets/sass/main.scss',
         '_dist/assets/_vulmix/css/main.vulmix.css'
@@ -68,15 +67,33 @@ class VulmixInit {
         },
       })
 
+      .after(stats => {
+        /**
+         * Only prints user files to the terminal
+         */
+        const assets = { ...stats.compilation.assets }
+        stats.compilation.assets = {}
+
+        for (const [path, asset] of Object.entries(assets)) {
+          if (!path.match(/((\.|_)vulmix|\.map)/)) {
+            stats.compilation.assets[path] = asset
+          }
+        }
+      })
+
+    if (fs.existsSync('assets/icons/')) {
+      mix.copy('assets/icons', '_dist/assets/icons')
+    }
+
     /**
-     * Production mode
+     * Production mode only
      */
     if (mix.inProduction()) {
       mix
         .copy('.vulmix/utils/deploy/.htaccess', '_dist')
     } else {
       /**
-       * Development mode
+       * Development mode only
        */
       mix
         .serve(

app.vue

@@ -6,6 +6,8 @@
 
     <meta name="title" content="Your application title">
     <meta name="description" content="Your application description">
+
+    <link rel="icon" href="/assets/icons/favicon.png">
   </Head>
 
   <App />

package.json

@@ -21,6 +21,7 @@
     "autoprefixer": "^10.4.8",
     "browser-sync": "^2.27.10",
     "browser-sync-webpack-plugin": "^2.3.0",
+    "fs": "^0.0.1-security",
     "laravel-mix": "^6.0.49",
     "laravel-mix-ejs": "^2.0.0",
     "laravel-mix-replace-in-file": "^0.1.0",