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
fix: 4.x gif thumb url base64 type should be corrected #44129
Conversation
Run & review this pull request in StackBlitz Codeflow. |
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## 4.x-stable #44129 +/- ##
============================================
Coverage 100.00% 100.00%
============================================
Files 450 450
Lines 8345 8351 +6
Branches 2540 2542 +2
============================================
+ Hits 8345 8351 +6
☔ View full report in Codecov by Sentry. |
@@ -303,7 +303,7 @@ exports[`MonthPicker and WeekPicker render WeekPicker 1`] = ` | |||
</thead> | |||
<tbody> | |||
<tr | |||
class="ant-picker-week-panel-row ant-picker-week-panel-row-selected" | |||
class="ant-picker-week-panel-row" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为啥这个会变?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
react-component/picker#664 貌似是这个,修复了 4.x 的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zombieJ 看看是否是预期行为。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
嗯,预期的。rc-picker patch 也是从 v5 里 pick 的
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#44083 这个 pr 修复到 4.x
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 55d5318
This pull request fixes some issues and adds some features related to the upload component. It prevents errors and memory leaks from object URLs in tests, adds a test case for gif preview, and improves the
fileToObject
function to handle gif files and release object URLs.🔍 Walkthrough
🤖 Generated by Copilot at 55d5318
window.URL.revokeObjectURL
in tests to avoid jsdom errors (link, link)fileToObjectUrl
to avoid memory leaks (link)fileToObjectUrl
by reading data URL directly without canvas (link)