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

fix: 修复ProFormSelect/LightSelect选中之后没有自动隐藏和fieldProps使用open属性不能效两个问题 #8035

Closed
wants to merge 1 commit into from

Conversation

gb853940223
Copy link
Contributor

此次PR修复了两个问题:
1、ProFormSelect选中某一项之后,下拉框没有自动消失。其实源码中有此逻辑,只不过没有生效,是因为下拉框选中自动消失的逻辑受到了其他代码的影响。我已经在此分支里加了相应的注释,并进行了修复。
2、在使用ProFormSelect组件时,在fieldProps中使用open属性,不生效。但是按照ProComponents官方文档的描述,fieldProps应该与antd中Select组件的props相同。导致源码逻辑与ProComponents官方文档的描述不一致。

Copy link

github-actions bot commented Jan 2, 2024

⚡️ Deploying PR Preview...

return restProps?.open
}
return open;
}, [open, restProps])

Check notice

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (91% of all statements in
the enclosing function
have an explicit semicolon).
Copy link

codecov bot commented Jan 7, 2024

Codecov Report

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

Comparison is base (a2be303) 97.59% compared to head (3029866) 97.58%.
Report is 2 commits behind head on master.

Files Patch % Lines
.../field/src/components/Select/LightSelect/index.tsx 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8035      +/-   ##
==========================================
- Coverage   97.59%   97.58%   -0.01%     
==========================================
  Files         299      299              
  Lines        9427     9430       +3     
  Branches     3503     3504       +1     
==========================================
+ Hits         9200     9202       +2     
- Misses        209      210       +1     
  Partials       18       18              

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

@gb853940223 gb853940223 closed this Jan 8, 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.

2 participants