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

site: optimization site style #48511

Closed
wants to merge 1 commit into from
Closed

site: optimization site style #48511

wants to merge 1 commit into from

Conversation

li-jia-nan
Copy link
Member

中文版模板 / Chinese template

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Workflow
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English site: optimization site style
🇨🇳 Chinese site: optimization site style

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

Copy link

stackblitz bot commented Apr 17, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link
Contributor

github-actions bot commented Apr 17, 2024

Preview is ready

Copy link
Contributor

github-actions bot commented Apr 17, 2024

👁 Visual Regression Report for PR #48511 Passed ✅

🎯 Target branch: master (0d34e0b)
📖 View Full Report ↗︎

🎊 Congrats! No visual-regression diff found.

Copy link
Contributor

size-limit report 📦

Path Size
./dist/antd.min.js 336.55 KB
./dist/antd-with-locales.min.js 383.85 KB

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link

codecov bot commented Apr 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (a512f81) to head (19b2585).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master    #48511   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          744       744           
  Lines        12882     12883    +1     
  Branches      3368      3369    +1     
=========================================
+ Hits         12882     12883    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@li-jia-nan li-jia-nan marked this pull request as ready for review April 17, 2024 11:01
@MadCcc
Copy link
Member

MadCcc commented Apr 17, 2024

这个解决什么问题?

@li-jia-nan
Copy link
Member Author

只保留数字,不显示完整链接:

image

@afc163
Copy link
Member

afc163 commented Apr 17, 2024

图片

线上貌似没问题,没看出有啥区别。

@li-jia-nan
Copy link
Member Author

li-jia-nan commented Apr 17, 2024

是这里有问题:

上面那个是我随便截的图

image

@MadCcc
Copy link
Member

MadCcc commented Apr 17, 2024

那不应该改的,github 对于其他仓库的 PR 也不会省略到只有 PR 号码

@li-jia-nan
Copy link
Member Author

那不应该改的,github 对于其他仓库的 PR 也不会省略到只有 PR 号码

那我先关了

@li-jia-nan li-jia-nan closed this Apr 17, 2024
@li-jia-nan li-jia-nan deleted the site-bug-reason branch April 17, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants