tdesign-react icon indicating copy to clipboard operation
tdesign-react copied to clipboard

[Popup] Popup 背景色和箭头的颜色不一致

Open linjunc opened this issue 2 years ago • 6 comments

tdesign-react 版本

1.1.2

重现链接

No response

重现步骤

popup 组件更改背景颜色,三角形颜色不变

期望结果

期望箭头颜色跟随浮层的颜色

实际结果

image

框架版本

react 18

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

linjunc avatar Jun 01 '23 05:06 linjunc

👋 @linjunc,感谢给 TDesign 提出了 issue。 请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

github-actions[bot] avatar Jun 01 '23 05:06 github-actions[bot]

♥️ 有劳 @jas0ncn @andyjxli @uyarn @carolin913 @honkinglin 尽快确认问题。 确认有效后将下一步计划和可能需要的时间回复给 @linjunc 。

github-actions[bot] avatar Jun 01 '23 05:06 github-actions[bot]

image 这个组件本来就没有暴露直接修改颜色的API, 但你也可以通过css选择器直接修改小箭头的背景色,是可以达到你的需求的。 这是我刚改的截图

Ali-ovo avatar Jun 09 '23 09:06 Ali-ovo

image 这个组件本来就没有暴露直接修改颜色的API, 但你也可以通过css选择器直接修改小箭头的背景色,是可以达到你的需求的。 这是我刚改的截图

是的,但是我理解这是一个 bug,而不是成为开发者的负担,竟然提供了 overlayStyle 这类的样式,我理解就应该处理好这件事情。而不是开发者再自行处理。

因此我觉得提供颜色 API 一并处理三角和背景颜色 更为合理

linjunc avatar Jun 09 '23 09:06 linjunc

image 这个组件本来就没有暴露直接修改颜色的API, 但你也可以通过css选择器直接修改小箭头的背景色,是可以达到你的需求的。 这是我刚改的截图

是的,但是我理解这是一个 bug,而不是成为开发者的负担,竟然提供了 overlayStyle 这类的样式,我理解就应该处理好这件事情。而不是开发者再自行处理。

因此我觉得提供颜色 API 一并处理三角和背景颜色 更为合理

这只是个通用组件库,专门为了背景颜色来提供一个API肯定是太现实的 例如这种情况: 如果我只想主体的背景颜色是一个颜色, 但是箭头的颜色是另外一种颜色, 那这样只有一个改颜色的API算不算有了另外一个bug

而且在组件内部是无法知道你这边传入的颜色是什么,为了方便使用者自定义这种样式,所以才提供了类名给使用者修改样式

Ali-ovo avatar Jun 09 '23 09:06 Ali-ovo

image 这个组件本来就没有暴露直接修改颜色的API, 但你也可以通过css选择器直接修改小箭头的背景色,是可以达到你的需求的。 这是我刚改的截图

是的,但是我理解这是一个 bug,而不是成为开发者的负担,竟然提供了 overlayStyle 这类的样式,我理解就应该处理好这件事情。而不是开发者再自行处理。

因此我觉得提供颜色 API 一并处理三角和背景颜色 更为合理

不过如果能提供的话确实是让使用者会有更多的选择,如果遇到这样的需求会方便一点

Ali-ovo avatar Jun 09 '23 10:06 Ali-ovo