Repository navigation
Conversation
|
@yunfeizhu is attempting to deploy a commit to the afc163's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
Walkthrough表格语义配置新增 ChangesHolder 语义支持
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~12 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant BodyGrid
participant BodyHolder
participant TableContext
participant StaticContext
participant BodyWrapper
BodyGrid->>BodyHolder: 将 BodyHolder 用作虚拟列表组件
BodyHolder->>TableContext: 读取 holder 类名和样式
BodyHolder->>StaticContext: 读取 body wrapper 配置
BodyHolder->>BodyWrapper: 传递合并后的类名、样式和 ref
BodyWrapper->>BodyGrid: holder 滚动时触发 onScroll
Suggested reviewers: Merge Risk: ⚪ Minimal · up to
|
Summary
Related to ant-design/ant-design#59485. This implements the rc-table part of the proposed
header.holder/body.holderAPI.When
scroll.yorstickyis enabled, the header and body have separate containers andcontentis no longer rendered. Neither container currently accepts semantic classes or styles.header.holderto the outer fixed/sticky header container.body.holderto the body scroll container, including the virtual-list holder and the empty-data fallback.contentslot.content,header.wrapper,body.wrapper, and fixed summary behavior unchanged.The virtual holder uses the virtual list's existing
componentextension point; no virtual-list API or dependency change is needed.Example
The same holder keys work with
VirtualTable, with or without rows.Verification
git diff --checkpassed.Follow-up
Ant Design will need its holder types, semantic demo/docs, and dependency version updated after this lands and is released. This PR deliberately leaves fixed summary semantics out of scope and does not close the Ant Design issue by itself.
Summary by CodeRabbit