search

arvinxx / dumi-theme-antd-style

dumi-theme for antd-style site
https://dumi-theme-antd-style.arvinx.com
MIT License
81 stars 42 forks source link

👑 [需求] contentMaxWidth 希望支持 string 类型，如 "100%" #12

Closed CJY0208 closed 1 year ago

CJY0208 commented 1 year ago

🥰 需求描述

目前 contentMaxWidth 仅支持数字，部分 demo 场景希望充分利用屏幕宽度

希望 contentMaxWidth 可调整至 100% 相对值

🧐 解决方案

🚑 其他信息

arvinxx commented 1 year ago

直接设成100%会有问题吗？除了类型定义出错以外

CJY0208 commented 1 year ago

用到了 contentMaxWidth 的 max-width 属性会失效，因为 contentMaxWidth 变量目前在 cssinjs 中的使用方式为 max-width: ${contentMaxWidth}px

会得到 max-width: 100%px

能够看到的影响就是，content 区域宽度固定变成了 960px

image

arvinxx commented 1 year ago

👌🏻 了解了，我处理下

CJY0208 commented 1 year ago

contentMaxWidth 我还有一个疑惑，目前 PC 状态下，contentMaxWidth 生效的条件是媒体查询 ${responsive.desktop} 也生效，否则只会使用 max-width: 960px

这个限制是否可以去掉？contentMaxWidth 默认值 1152 和 960 相差也不大，不太理解为啥要做这个限制

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 0.19.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

arvinxx commented 1 year ago

contentMaxWidth 我还有一个疑惑，目前 PC 状态下，contentMaxWidth 生效的条件是媒体查询 ${responsive.desktop} 也生效，否则只会使用 max-width: 960px

这个限制是否可以去掉？contentMaxWidth 默认值 1152 和 960 相差也不大，不太理解为啥要做这个限制

960 是1440 (mbp)下的限制， 1152 是 1920 (pc）下的限制。

我们做过最佳视域的研究，这基本上是 pc 下默认最舒服的尺寸。