Skip to content

Conversation

@keeplearning66
Copy link
Contributor

@keeplearning66 keeplearning66 commented Jul 14, 2025

This PR is to fix #13492

The bug is caused by the slot change leading to an update, but the input field value is not set.

I listen to the update caused by the slot change through onUpdated and call updateValue to set the input field value.

After modification

20250723_191405.mp4

I’m sorry, I’m not sure how to write a unit test for this scenario.

@codecov-commenter
Copy link

codecov-commenter commented Jul 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.80%. Comparing base (ec5b45b) to head (062ede3).
⚠️ Report is 133 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13559      +/-   ##
==========================================
+ Coverage   89.60%   89.80%   +0.20%     
==========================================
  Files         257      257              
  Lines        7013     7045      +32     
  Branches     1736     1743       +7     
==========================================
+ Hits         6284     6327      +43     
+ Misses        384      379       -5     
+ Partials      345      339       -6     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@keeplearning66 keeplearning66 changed the title fix(Field): set the value of the input field when the slot change causes an update fix(Field): set value of the input field when slot change causes an update Jul 18, 2025
@code-farmer-i
Copy link

When will this problem be fixed?

@keeplearning66
Copy link
Contributor Author

I'm not sure either — this PR still hasn't been reviewed. 😢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug Report] 当我尝试动态设置 Field 组件的 input 插槽时,输入框的回显看起来有点问题

3 participants