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.

Sign up for GitHub

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

Jump to bottom

[Rate] 评分图标间距不对 #117

Closed
Wen1kang opened this issue Apr 27, 2022 · 7 comments · Fixed by #149, Tencent/tdesign-common#536 or #208
Closed

[Rate] 评分图标间距不对 #117

Wen1kang opened this issue Apr 27, 2022 · 7 comments · Fixed by #149, Tencent/tdesign-common#536 or #208

Comments

@Wen1kang
Copy link
Collaborator

tdesign-mobile-react 版本

内测版

重现链接

No response

重现步骤

No response

期望结果

icon与icon间距为8px，最后一个无间距

image

实际结果

间距为4px，最后一个还带margin

image

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response
@github-actions
Copy link
Contributor

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

@Perisiguiendo
Copy link
Contributor

目前设计稿尺寸为
image

Perisiguiendo added a commit to Perisiguiendo/tdesign-mobile-react that referenced this issue May 19, 2022
@Perisiguiendo
fix(rate): add disable methods & fix styles problem & fix half rate
03a368b 
resolve Tencent#122, Tencent#121, Tencent#118, Tencent#117
@Perisiguiendo Perisiguiendo mentioned this issue May 19, 2022
Merged
16 tasks
@Wen1kang
Copy link
Collaborator Author

@Perisiguiendo 这个设计稿是哪里的呀，我看原稿是8px

@Perisiguiendo
Copy link
Contributor

Perisiguiendo commented Jun 10, 2022 •
edited
Loading

@Wen1kang https://codesign.woa.com/app/design/NlO1ZnV4YJZLMV2/BGAE9KxJJV0lRd8/inspect
抱歉没有收到邮件，是这个链接，辛苦看一下

@Wen1kang
Copy link
Collaborator Author

@Perisiguiendo 这个很旧设计稿，会有问题，我重新上传一下

@Wen1kang
Copy link
Collaborator Author

@Perisiguiendo 已经重新上传了，麻烦看看哦【腾讯 CoDesign】TDesign移动端组件设计稿
https://codesign.woa.com/s/JG2mj75B3G0VKdM/2bzpZvnkrnZkAaV/inspect

@Perisiguiendo
Copy link
Contributor

@Wen1kang 收到，辛苦哈

@LeeJim LeeJim linked a pull request Jun 14, 2022 that will close this issue
修复 Rate 样式问题 #149
Merged
16 tasks
@LeeJim LeeJim mentioned this issue Jun 16, 2022
Merged
16 tasks
@LeeJim LeeJim mentioned this issue Jun 16, 2022
Merged
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@Perisiguiendo @Wen1kang