Yiheng

Results 133 comments of Yiheng

> > 一般做动态主题时,会给 html 加上特定类名(如 `.dark`)或者特定属性(`data-theme="dark"`) 等,来实现动态换肤,所以这个 hook 应该还需要加个回调参数 `onChange`,这样 theme 变动时,用户可以自己完成上述这些操作。@ianzone 你觉着呢? > > 代码我更新了一部分,你改之前 pull 一下 > > 这个 onChange 是不是也可以不加呢,直接监听 theme 的变化就好了? 赞同

> 另外这个 hook 叫 useTheme 是不是不太合适呢,看到这个 hook 的名称无法很明确的知道这个 hook 干什么的,这个跟 darkMode 比较相关,是不是可以考虑 hook 的名字结合一下这个 但是这个hooks包括了light, dark, system 三种mode,而且确实和主题相关,所以我认为命名合适

没问题的话可否合并一下?我需要用这个钩子。

Yeah, just trying to find a shortcut, rather than setting 3 variables.

I just added the test.

Hi, is it ok?

尝试了一下,代码逻辑比较难懂,然后运行都一堆报错,放弃。 思路是在 [ThemeAntVContext.Provider](https://github.com/antvis/dumi-theme-antv/blob/v3/src/plugin/index.ts) 里嵌套 [ConfigProvider](https://ant-design.antgroup.com/components/config-provider-cn?theme=dark),然后控制 ConfigProvider 的 theme 属性。