search

emosheeep / vite-plugin-virtual-mpa

Out-of-box MPA plugin for Vite, generate multiple entries using only one template.
https://stackblitz.com/~/github.com/emosheeep/vite-plugin-virtual-mpa
MIT License
120 stars 15 forks source link

feat(plugin): integrate `html-minifier-terser` #18

Closed jiadesen closed 1 year ago

jiadesen commented 1 year ago

集成 html 混淆

emosheeep commented 1 year ago

这个是出于什么考虑呢，想压缩html的话可以单独使用这个插件，不需要强制集成进来呀

jiadesen commented 1 year ago

对于项目，能省一个依赖是不是也不错...

emosheeep commented 1 year ago

非常感谢你对这个插件功能拓展做出的贡献，不过作为插件作者的话，我认为这个插件的主要的主要目的和初衷是提供一套相对完善的mpa通用配置方案，他不是一个定制插件，html压缩功能我个人觉得可能，不是很相关，加进来的话不利于插件后续的功能维护。 另外有可能其他用户不需要该功能，只是单纯想使用mpa的解决方案来着，绑定使用方式可能会带来额外的心智负担。

jiadesen commented 1 year ago

那加个开关，默认不压缩？这个功能集成进来也很合理啊，就像支持 ejs 一样，用起来也更接近 webpack。参考 vite-plugin-html，我们也就差一个 HTML 压缩能力。 其实我认为对于大部分 vue 项目，入口 html 即使不压缩也没关系（体积能差1kb...），但是就有挑刺的说跟 webpack 不一致，构建的 html 没压缩（mmp...），为此不得不再找个压缩插件

jiadesen commented 1 year ago

绑定使用方式可能会带来额外的心智负担

所以我给了 minify 一些默认配置，绝大部分情况不需要干预

emosheeep commented 1 year ago

那个插件我看过，是vban那个admin里面的插件吧，我有空再看看。

emosheeep commented 1 year ago

为啥会有人挑刺😂😂

jiadesen commented 1 year ago

一个字 菜...

emosheeep commented 1 year ago

还不回家过年吗？我这两天旅游呢🤣🤣，暂时没空看哦

jiadesen commented 1 year ago

午休时间都休息了，我没这习惯，要么玩手机要么看电脑... 不着急，最后一点事情，已经改完了，新年快乐~

jiadesen commented 1 year ago

冲突不好解，一会儿重新搞一下