Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
linjunc opened this issue Jun 1, 2023 · 6 comments
Open

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

linjunc opened this issue Jun 1, 2023 · 6 comments
Assignees
Labels
🎨 design Something about design or style enhancement New feature or request

Comments

@linjunc
Copy link
Contributor

linjunc commented Jun 1, 2023

tdesign-react 版本

1.1.2

重现链接

No response

重现步骤

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

期望结果

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

实际结果

image

框架版本

react 18

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2023

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

@linjunc linjunc changed the title [组件名称] 描述问题的标题 [Popup] Popup 背景色和箭头的颜色不一致 Jun 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2023

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

@Ali-ovo
Copy link
Contributor

Ali-ovo commented Jun 9, 2023

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

@linjunc
Copy link
Contributor Author

linjunc commented Jun 9, 2023

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

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

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

@Ali-ovo
Copy link
Contributor

Ali-ovo commented Jun 9, 2023

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

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

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

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

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

@Ali-ovo
Copy link
Contributor

Ali-ovo commented Jun 9, 2023

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

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

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

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

@HaixingOoO HaixingOoO added enhancement New feature or request 🎨 design Something about design or style labels Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 design Something about design or style enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants