Skip to content

Comments

fix(datepicker): 修复在使用 DatePicker 时页面会短暂出现 0 的问题#3399

Merged
irisSong merged 1 commit intojdf2e:feat_v3.xfrom
shincurry:fix-datepicker
Dec 23, 2025
Merged

fix(datepicker): 修复在使用 DatePicker 时页面会短暂出现 0 的问题#3399
irisSong merged 1 commit intojdf2e:feat_v3.xfrom
shincurry:fix-datepicker

Conversation

@shincurry
Copy link
Contributor

@shincurry shincurry commented Dec 4, 2025

🤔 这个变动的性质是?

  • 新特性提交
  • 日常 bug 修复
  • 站点、文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • TypeScript 定义更新
  • 包体积优化
  • 性能优化
  • 功能增强
  • 国际化改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他改动(是关于什么的改动?)

🔗 相关 Issue

💡 需求背景和解决方案

DatePicker 组件定义了 pickerOptions 状态,初始化为空数组,然后使用 useEffect 来生成真正的 Options。
在较短的时间内 pickerOptions.length 为 0,导致 {pickerOptions.length && (/** node */) 这样的写法会在 DOM 中短暂渲染出 0

改为 {pickerOptions.length > 0 && (/** node */) 即可

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • fork仓库代码是否为最新避免文件冲突
  • Files changed 没有 package.json lock 等无关文件

Summary by CodeRabbit

发布说明

  • 重构
    • 优化日期选择器组件的条件判断逻辑,增强代码清晰度和可读性。

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 4, 2025

概述

该拉取请求将日期选择器组件中的真假值检查替换为显式长度比较,从 {pickerOptions.length} 改为 {pickerOptions.length > 0},提高代码的可读性和意图的明确性。

变更

文件夹 / 文件 变更摘要
日期选择器条件渲染
src/packages/datepicker/datepicker.tsx
将选择器选项的真假值检查替换为显式非空比较,从隐式的真假判断改为明确的大于零比较,确保条件判断的意图明确。

代码审查估计工作量

🎯 1 (微小改动) | ⏱️ ~3 分钟

  • 仅涉及单个文件中的单行变更
  • 属于代码风格和可读性改进,无业务逻辑变动

可能相关的拉取请求

建议审查者

  • xiaoyatong
  • oasis-cloud

诗歌

🐰 一个小小的改变,微妙而精细,
真假变比较,逻辑更明晰。
代码如篇章,每字都有意,
小兔拍拍手,祝贺这次修缮!✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 标题清晰准确地描述了主要变更:修复DatePicker组件在初始化时页面短暂出现0的bug问题。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed PR描述基本完整,正确标记为bug修复,清晰说明问题(pickerOptions长度为0时DOM渲染出0)和解决方案,自检清单全部勾选。
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added action:review This PR needs more reviews (less than 2 approvals) 3.x Target branch 3.x labels Dec 4, 2025
@irisSong irisSong merged commit da29016 into jdf2e:feat_v3.x Dec 23, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x Target branch 3.x action:review This PR needs more reviews (less than 2 approvals) size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants