serveIndex: set icons by default.

[jlguenego]

Change Request :
the serveIndex currently accepts icons. WebpackDevServer would be a nicer app if icons were visible by default.

Yes it is very minor.

[alexander-akait]

Do you want setup own icons or we should setup default icon?

[jlguenego]

Just adding {icons:true} to serveIndex.

Here is the code change I would suggest in the lib/Server.js:

setupServeIndexFeature() {
    const contentBase = this.options.contentBase;
    const contentBasePublicPath = this.options.contentBasePublicPath;

    if (Array.isArray(contentBase)) {
      contentBase.forEach((item) => {
        this.app.use(contentBasePublicPath, (req, res, next) => {
          // serve-index doesn't fallthrough non-get/head request to next middleware
          if (req.method !== 'GET' && req.method !== 'HEAD') {
            return next();
          }

          serveIndex(item, { icons: true })(req, res, next);
        });
      });
    } else if (
      typeof contentBase !== 'number' &&
      !isAbsoluteUrl(String(contentBase))
    ) {
      this.app.use(contentBasePublicPath, (req, res, next) => {
        // serve-index doesn't fallthrough non-get/head request to next middleware
        if (req.method !== 'GET' && req.method !== 'HEAD') {
          return next();
        }

        serveIndex(contentBase, { icons: true })(req, res, next);
      });
    }
  }

[alexander-akait]

Solved