hellof2e / quark-design

Browser-native component library, framework-agnostic, base on web components.(移动端 UI 组件库,无框架,即插即用。)
https://quark-ecosystem.github.io/quarkd-docs
MIT License
1.9k stars 189 forks source link

global-css.js 的变量似乎没有全部 CSS Variables 化 #237

Closed yoyo837 closed 1 year ago

yoyo837 commented 1 year ago

比如 quark-fontSizeBasequark-textSecondaryColor

:host .quark-actionsheet-title {
  color: quark-textSecondaryColor;
  font-size: quark-fontSizeBase;
  display: flex;
  height: 60px;
  width: 100%;
  justify-content: center;
  align-items: center;
  border-bottom: 0.5px solid #ebedf0;
  text-align: center;
}

请问是有意的还是遗漏? 遗漏的话我发个PR补上来.

sanqi-med commented 1 year ago

具体什么意思还没太明白,你是指0.5px, #ebedf0 这种没有 Variable 吗? 我们目前只针对通用性的做了Variable,个性化的都直接写具体的值了

yoyo837 commented 1 year ago

比如上面示例的font-size: quark-fontSizeBase;, 是不是也应该var一个变量? font-size: var(--xxxxxxx. quark-fontSizeBase); 这个意思.

yoyo837 commented 1 year ago

所有使用 global-css.js 里值的地方

sanqi-med commented 1 year ago

是不是也应该var一个变量? font-size: var(--xxxxxxx. quark-fontSizeBase); 这个意思.

这个要看具体的需求,有些是故意不暴露变量的,因为有些样式是统一的,不给修改的

yoyo837 commented 1 year ago

那是否可以理解为, 并不存在 没有全部 CSS Variables 化 的问题, 没有写 var 的就都是故意的不可定制的位置?

sanqi-med commented 1 year ago

那是否可以理解为, 并不存在“没有全部 CSS Variables 化”的问题, 没有写var的就都是故意的不可定制的位置?

可以这么理解

yoyo837 commented 1 year ago

感谢回复!