修复打包导致的 ImageUploader 中 Space 和 Grid 样式的权重问题以及css变量作用域导致的 --gap-horizontal 和 --gap-vertical 变量未正常生效的问题
#6440
Conversation
cadleo
changed the title
ImageUploader 中 Space 和 Grid 的样式权重问题及 --gap-horizontal 和 --gap-vertical 变量未正常生效的问题ImageUploader 中 Space 和 Grid 样式的权重问题以及css变量作用域导致的 --gap-horizontal 和 --gap-vertical 变量未正常生效的问题
| ---gap: var(--gap); | ||
| ---gap-horizontal: var(--gap-horizontal, var(--gap)); | ||
| ---gap-vertical: var(--gap-vertical, var(--gap)); | ||
| ---gap-horizontal: var(--gap-horizontal); |
There was a problem hiding this comment.
自循环写法感觉很怪异，包括以前的写法也很怪。
确实比较奇怪，但个人暂未想到其他办法，除非在ts文件里把css变量pick出来，丢到 Space 和 Grid 组件上
There was a problem hiding this comment.
从代码看之前就是想用外面配置的 ---gap-xxx 做传递。我感觉是换个变量名字会更好一些。
There was a problem hiding this comment.
从代码看之前就是想用外面配置的 ---gap-xxx 做传递。我感觉是换个变量名字会更好一些。
不建议改，一来向前兼容，二来能保持变量名与space和grid相同，降低理解成本
There was a problem hiding this comment.
那就不好搞了，--gap-horizontal 有了用 --gap-horizontal，没有才用 --gap 是预期行为。外部提供了 --gap-horizontal 也是“符合预期”的。
There was a problem hiding this comment.
那就不好搞了，
--gap-horizontal有了用--gap-horizontal，没有才用--gap是预期行为。外部提供了--gap-horizontal也是“符合预期”的。
所以也就俩选项了：用当前比较奇怪的css写法，或者写到ts里，pick之后放到space和grid上
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6440 +/- ##
=======================================
Coverage 92.10% 92.10%
=======================================
Files 316 316
Lines 6861 6861
Branches 1719 1719
=======================================
Hits 6319 6319
Misses 504 504
Partials 38 38 ☔ View full report in Codecov by Sentry. |
|
PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-6440.surge.sh |
antd-mobile的项目在打包时，由于分包的原因，会导致如下问题，故需要修改样式权重：