Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(webpack5-runner): 修复 imageUrlLoaderOption 配置项失效问题 #13014

Merged
merged 5 commits into from
Jan 17, 2023

Conversation

beezen
Copy link
Contributor

@beezen beezen commented Dec 23, 2022

这个 PR 做了什么? (简要描述所做更改)

修复 imageUrlLoaderOption 配置项失效问题。

【详细说明】

升级到webpack5之后,我们提供了 mini.imageUrlLoaderOption 和 h5.imageUrlLoaderOption 这样的配置来设置针对图片的 url-loader 的配置。

但是,这个配置目前是不生效的,原因是我们以前的 postcss-url 插件中默认配置是设置了 url inline 模式,即所有的图片已经被内联为 base64了,这就导致在 imageUrlLoaderOption 配置解析时,无法匹配到图片路径格式,即功能失效。

因为两边的配置功能基本雷同,建议默认关闭 postcss-url 中的设置,同时也是因为 postcss-url 的配置也没有在文档中对外公开。

这个 PR 是什么类型? (至少选择一个)

  • 错误修复(Bugfix) issue: fix #
  • 新功能(Feature)
  • 代码重构(Refactor)
  • TypeScript 类型定义修改(Typings)
  • 文档修改(Docs)
  • 代码风格更新(Code style update)
  • 其他,请描述(Other, please describe):

这个 PR 涉及以下平台:

  • 所有小程序
  • 微信小程序
  • 支付宝小程序
  • 百度小程序
  • 字节跳动小程序
  • QQ 轻应用
  • 京东小程序
  • 快应用平台(QuickApp)
  • Web 平台(H5)
  • 移动端(React-Native)

@taro-bot2
Copy link

taro-bot2 bot commented Dec 23, 2022

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

2 similar comments
@taro-bot2
Copy link

taro-bot2 bot commented Dec 23, 2022

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

@taro-bot2
Copy link

taro-bot2 bot commented Dec 23, 2022

欢迎提交 PR~ Taro 非常感谢您对开源事业做出的贡献!🌷🌷🌷

一般 PR 会在一到两周内进行 review,成功合入后会随下一个版本进行发布。

Review 需要耗费大量时间,所以请遵循以下规范,协助我们提高 review 效率🙏🙏🙏

  1. 详细介绍 PR 的背景(非常重要,例如解决了什么问题,该问题如何复现等)
  2. 确保 CI 顺利运行。
  3. 最好能提供对应的测试用例。

为了更好地进行沟通,请加入 Taro 开发者微信群:

@Chen-jj
Copy link
Contributor

Chen-jj commented Dec 27, 2022

@beezen 印象中小程序的 background-image 只能支持 base64 的样子,这样的话小程序的 postcss-url 还是有必要的,可以测试下~

@beezen beezen closed this Dec 27, 2022
@beezen beezen reopened this Dec 27, 2022
@beezen
Copy link
Contributor Author

beezen commented Dec 27, 2022

@Chen-jj 奥奥 之前确实忽略了。 小程序端确实是还得保留默认的 inline 设置 ; 不过因为不同的小程序端底层的处理方式有区别,有些小程序其实能够支持本地路径的调用,现在这个逻辑对开发者来说有一点黑盒,感觉可以在文档那边添加一个 postcss.url 配置说明。

@Chen-jj
Copy link
Contributor

Chen-jj commented Jan 17, 2023

感谢贡献~

@Chen-jj Chen-jj merged commit a311d11 into NervJS:next Jan 17, 2023
@Chen-jj
Copy link
Contributor

Chen-jj commented Feb 16, 2023

jdf2e/nutui#2089

我们改的还是不对,postcss-url 插件最重要的用途是修正 @import 的资源路径,是一定需要开启的。应该将 inline 模式改为 rebase 模式,然后交给 webpack asset module 处理。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants