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

⚡ feat: use memo wrapper in markdown #125

Merged
merged 2 commits into from
Dec 22, 2023
Merged

⚡ feat: use memo wrapper in markdown #125

merged 2 commits into from
Dec 22, 2023

Conversation

ONLY-yours
Copy link
Collaborator

在单独用这个组件的时候不明显,但是一到流式输出的时候就会发现 dom 一直在重新渲染,一定得 memo 包裹下,不然长的时候渲染性能爆炸

Copy link

github-actions bot commented Dec 22, 2023

🎊 PR Preview 7c7382f has been successfully built and deployed to https://ant-design-pro-editor-preview-pr-125.surge.sh

🕐 Build time: 74.022s

🤖 By surge-preview

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b96b378) 81.51% compared to head (19b310b) 81.51%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #125   +/-   ##
=======================================
  Coverage   81.51%   81.51%           
=======================================
  Files         432      432           
  Lines       26886    26891    +5     
  Branches     1232     1233    +1     
=======================================
+ Hits        21916    21921    +5     
  Misses       4970     4970           

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

Copy link
Member

@rdmclin2 rdmclin2 left a comment

Choose a reason for hiding this comment

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

+1

@rdmclin2 rdmclin2 merged commit 01049aa into main Dec 22, 2023
2 checks passed
@rdmclin2 rdmclin2 deleted the memo/markdown branch December 22, 2023 09:19
Copy link

🎉 This PR is included in version 0.34.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants