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

refactor: the implementation of dragger #3647

Merged

Conversation

zxyao145
Copy link
Member

🤔 This is a ...

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

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English refactor: the implementation of dragger
🇨🇳 Chinese 修改 Dragger 的实现方法

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

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

Copy link

github-actions bot commented Jan 22, 2024

Copy link

codecov bot commented Jan 30, 2024

Codecov Report

Attention: 131 lines in your changes are missing coverage. Please review.

Comparison is base (bdc1e4f) 43.59% compared to head (bf1038b) 43.57%.
Report is 4 commits behind head on master.

Files Patch % Lines
...omponents/core/JsInterop/modules/dom/dragHelper.ts 0.00% 129 Missing ⚠️
components/select/Select.razor 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3647      +/-   ##
==========================================
- Coverage   43.59%   43.57%   -0.03%     
==========================================
  Files         572      572              
  Lines       27634    27649      +15     
  Branches     5621     5622       +1     
==========================================
+ Hits        12046    12047       +1     
- Misses      14592    14607      +15     
+ Partials      996      995       -1     

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

@ElderJames ElderJames merged commit 84c3c5f into ant-design-blazor:master Jan 31, 2024
4 of 6 checks passed
Magehernan pushed a commit to GestorMax/ant-design-blazor that referenced this pull request Feb 2, 2024
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

2 participants