Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: popover component & new topic actions layout #580

Merged
merged 10 commits into from
May 14, 2023

Conversation

y-young
Copy link
Contributor

@y-young y-young commented May 11, 2023

#112

添加 Popover 组件,按照设计稿修改小组讨论按钮,顺带把按钮封装成了组件

Popover 的测试不知道怎么搞,测试框架好像获取不到完整的 CSS 样式,试了 window.getComputeStyletoHaveStyle 都不行
现在 E2E 测试已经覆盖了,那就不写了吧(

@github-actions
Copy link
Contributor

github-actions bot commented May 11, 2023

Preview Deployment

Build URL time
Sites netlify preview 2023-04-00 15:37:04Z
Storybook netlify preview 2023-04-00 15:35:44Z

@mergify mergify bot added waiting-approval waiting code reviewers to approval PR and removed ci:test:e2e:fail labels May 11, 2023
@mergify mergify bot added ci:test:fail and removed waiting-approval waiting code reviewers to approval PR labels May 11, 2023
@mergify mergify bot added waiting-approval waiting code reviewers to approval PR and removed ci:test:fail labels May 11, 2023
@bangumi bangumi deleted a comment from codecov bot May 11, 2023
@codecov
Copy link

codecov bot commented May 12, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.23 🎉

Comparison is base (982593c) 82.46% compared to head (a525b3f) 82.69%.

❗ Current head a525b3f differs from pull request most recent head 4da9877. Consider uploading reports for the commit 4da9877 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #580      +/-   ##
==========================================
+ Coverage   82.46%   82.69%   +0.23%     
==========================================
  Files          98      100       +2     
  Lines        5953     6034      +81     
  Branches      421      424       +3     
==========================================
+ Hits         4909     4990      +81     
  Misses       1038     1038              
  Partials        6        6              
Impacted Files Coverage Δ
packages/design/components/Popover/index.tsx 100.00% <100.00%> (ø)
packages/design/components/Topic/Comment.tsx 91.88% <100.00%> (-0.37%) ⬇️
...ackages/design/components/Topic/CommentActions.tsx 100.00% <100.00%> (ø)
packages/design/components/Topic/CommentInfo.tsx 91.42% <100.00%> (-0.47%) ⬇️
packages/design/components/Topic/index.tsx 100.00% <100.00%> (ø)
packages/design/components/Topic/style/index.tsx 100.00% <100.00%> (ø)
packages/design/index.tsx 100.00% <100.00%> (ø)
packages/icons/index.tsx 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

</>
)}
{/* TODO: 实现绝交和报告疑虑功能 */}
<Button type='text'>绝交</Button>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

绝交是啥功能 (bgm38) (不是代码问题,纯好奇)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

啊?就是原站的拉黑功能啊

https://bgm.tv/settings/privacy

与以下用户绝交 · 不再看到用户的所有话题、评论、日志、私信、提醒

@cokemine
Copy link
Contributor

没有登录状态下只有楼层和发帖日期是不是最后不应该有分隔的竖线|。设计上好像没有体现

@trim21 trim21 merged commit 7ce2f19 into bangumi:master May 14, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
size/L waiting-approval waiting code reviewers to approval PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants