-
-
Notifications
You must be signed in to change notification settings - Fork 611
fix: stack case #1247
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: stack case #1247
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Walkthrough本次变更更新了表格示例中列宽和固定属性的配置,调整了部分列的宽度以及固定属性设置。同时,在 Cell 组件中新增了用于控制固定阴影偏移的属性,并在辅助函数与接口中增加了对应的列宽信息。这些改动均不改变已有公开实体的声明,而是扩展了功能配置和阴影计算逻辑。 Changes
Sequence Diagram(s)sequenceDiagram
participant U as 用户
participant T as 表格组件
participant S as useStickyOffsets Hook
participant C as Cell 组件
participant F as fixUtil 工具
U->>T: 滚动表格
T->>S: 提供列宽信息,计算粘性偏移
S-->>T: 返回 {start, end, widths}
T->>C: 传递 Cell 属性(包含偏移量)
C->>F: 计算固定阴影偏移
F-->>C: 返回计算后的 offsetFixedStartShadow/offsetFixedEndShadow
C->>T: 渲染带阴影效果的单元格
Possibly related PRs
Suggested reviewers
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure. 🔧 ESLint
docs/examples/scrollXY.tsxOops! Something went wrong! :( ESLint: 8.57.1 ESLint couldn't find the config "prettier" to extend from. Please check that the name of the config is correct. The config "prettier" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. src/Cell/index.tsxOops! Something went wrong! :( ESLint: 8.57.1 ESLint couldn't find the config "prettier" to extend from. Please check that the name of the config is correct. The config "prettier" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. src/hooks/useStickyOffsets.tsOops! Something went wrong! :( ESLint: 8.57.1 ESLint couldn't find the config "prettier" to extend from. Please check that the name of the config is correct. The config "prettier" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team.
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (5)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🔇 Additional comments (10)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1247 +/- ##
=======================================
Coverage 98.02% 98.03%
=======================================
Files 76 76
Lines 7391 7412 +21
Branches 1138 1145 +7
=======================================
+ Hits 7245 7266 +21
Misses 140 140
Partials 6 6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary by CodeRabbit
新特性
重构