Skip to content

Conversation

@irisSong
Copy link
Collaborator

@irisSong irisSong commented Jun 4, 2024

🤔 这个变动的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

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

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

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

Summary by CodeRabbit

  • 新功能

    • CountDown 组件添加了新的 type 属性,支持 defaultprimary 类型。
    • CountDown 组件中添加了 CountDownType 枚举和相关逻辑。
  • 样式

    • 调整了 CountDown 组件的颜色、字体大小、对齐和布局属性。
    • 更新了 CSS 变量,增加了对新样式属性的支持,包括字体粗细、宽度、高度、填充、边距、边框半径和背景颜色等。
  • 文档

    • 更新了中英双语文档,添加了新的 type 属性说明和相关样式变量的描述。

@oasis-cloud
Copy link
Collaborator

image
RN 下倒计时字体会导致展示跳动,可以给数字框加个最小宽度限制一下?

})

// TODO: 后期统一替换
const harmonyAndRn = [
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

可以合并下代码,直接把这里替换一下

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

可以合并下代码,直接把这里替换一下

已修改

height: pxTransform(25),
fontSize: pxTransform(14),
borderRadius: pxTransform(6),
height: pxTransform(20),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码可以合并一下 dev-harmony , 使用 uitls 中的 pxTransform

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码可以合并一下 dev-harmony , 使用 uitls 中的 pxTransform
已修改

@coderabbitai
Copy link

coderabbitai bot commented Jun 19, 2024

Warning

Rate limit exceeded

@irisSong has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 36 minutes and 22 seconds before requesting another review.

How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Commits

Files that changed from the base of the PR and between 0803528 and 6d7e0d9.

Walkthrough

此次更新主要集中在NutUI的倒计时组件和构建配置文件。这次更新包括对倒计时组件的样式进行优化,引入新的属性以增强自定义能力,同时更新了构建文件以标准化配置格式。

Changes

文件路径 变更摘要
packages/nutui-harmony/build-profile.json5 更新配置文件,使用字符串键定义不同字段。
src/packages/countdown/countdown.harmony.css, countdown.scss 优化倒计时组件样式,调整颜色、字体大小、对齐和布局属性。
countdown.taro.tsx, countdown.tsx 增加CountDownType枚举类型和type属性,更新时间格式化和渲染逻辑。
demo1.tsx, demo8.tsx (多个demo文件) 增加新的倒计时示例,调整样式和属性展示。
doc.en-US.md, doc.md, doc.zh-TW.md 更新倒计时组件文档,添加并调整组件属性和样式变量说明。
src/styles/variables.scss 添加新的样式变量,调整倒计时组件默认样式。

Poem

小兔提笔画倒计时,
样式多变更精致,
属性新添灵活用,
标准格式来助力。
组件更新迎新日,
一秒一刻皆美丽。


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

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 as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Comment on lines 760 to 797
$countdown-color: var(--nutui-countdown-color, $color-primary) !default;
$countdown-font-size: var(--nutui-countdown-font-size, 10px) !default;
$coutdown-font-weight: var(--nutui-countdown-font-weight, 400) !default;
$countdown-height: var(--nutui-countdown-height, 14px) !default;
$countdown-width: var(--nutui-countdown-width, 14px) !default;
$countdown-number-padding: var(
--nutui-countdown-number-padding,
0 1px
) !default;
$countdown-number-border-radius: var(
--nutui-countdown-number-border-radius,
2px
) !default;
$countdown-number-margin: var(--nutui-countdown-number-margin, 0 2px) !default;
$countdown-number-border-color: var(
--nutui-countdown-number-border-color,
$color-primary-light
) !default;
$countdown-number-color: var(
--nutui-countdown-number-color,
$color-primary
) !default;
$countdown-number-background-color: var(
--nutui-countdown-number-background-color,
transparent
) !default;
$countdown-number-primary-border-color: var(
--nutui-countdown-number-primary-border-color,
$color-primary
) !default;
$countdown-number-primary-color: var(
--nutui-countdown-number-primary-color,
#ffffff
) !default;
$countdown-number-primary-background-color: var(
--nutui-countdown-number-primary-background-color,
$color-primary
) !default;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

变量 $coutdown-font-weight 名称拼写错误,应为 $countdown-font-weight

- $coutdown-font-weight: var(--nutui-countdown-font-weight, 400) !default;
+ $countdown-font-weight: var(--nutui-countdown-font-weight, 400) !default;
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
$countdown-color: var(--nutui-countdown-color, $color-primary) !default;
$countdown-font-size: var(--nutui-countdown-font-size, 10px) !default;
$coutdown-font-weight: var(--nutui-countdown-font-weight, 400) !default;
$countdown-height: var(--nutui-countdown-height, 14px) !default;
$countdown-width: var(--nutui-countdown-width, 14px) !default;
$countdown-number-padding: var(
--nutui-countdown-number-padding,
0 1px
) !default;
$countdown-number-border-radius: var(
--nutui-countdown-number-border-radius,
2px
) !default;
$countdown-number-margin: var(--nutui-countdown-number-margin, 0 2px) !default;
$countdown-number-border-color: var(
--nutui-countdown-number-border-color,
$color-primary-light
) !default;
$countdown-number-color: var(
--nutui-countdown-number-color,
$color-primary
) !default;
$countdown-number-background-color: var(
--nutui-countdown-number-background-color,
transparent
) !default;
$countdown-number-primary-border-color: var(
--nutui-countdown-number-primary-border-color,
$color-primary
) !default;
$countdown-number-primary-color: var(
--nutui-countdown-number-primary-color,
#ffffff
) !default;
$countdown-number-primary-background-color: var(
--nutui-countdown-number-primary-background-color,
$color-primary
) !default;
$countdown-color: var(--nutui-countdown-color, $color-primary) !default;
$countdown-font-size: var(--nutui-countdown-font-size, 10px) !default;
$countdown-font-weight: var(--nutui-countdown-font-weight, 400) !default;
$countdown-height: var(--nutui-countdown-height, 14px) !default;
$countdown-width: var(--nutui-countdown-width, 14px) !default;
$countdown-number-padding: var(
--nutui-countdown-number-padding,
0 1px
) !default;
$countdown-number-border-radius: var(
--nutui-countdown-number-border-radius,
2px
) !default;
$countdown-number-margin: var(--nutui-countdown-number-margin, 0 2px) !default;
$countdown-number-border-color: var(
--nutui-countdown-number-border-color,
$color-primary-light
) !default;
$countdown-number-color: var(
--nutui-countdown-number-color,
$color-primary
) !default;
$countdown-number-background-color: var(
--nutui-countdown-number-background-color,
transparent
) !default;
$countdown-number-primary-border-color: var(
--nutui-countdown-number-primary-border-color,
$color-primary
) !default;
$countdown-number-primary-color: var(
--nutui-countdown-number-primary-color,
#ffffff
) !default;
$countdown-number-primary-background-color: var(
--nutui-countdown-number-primary-background-color,
$color-primary
) !default;

@irisSong
Copy link
Collaborator Author

image RN 下倒计时字体会导致展示跳动,可以给数字框加个最小宽度限制一下?

已修改

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 4

Comment on lines 138 to 151
| \--nutui-countdown-color | 倒計時的文字顔色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒計時的字體大小 | `10px` |
| \--nutui-countdown-font-weight | 倒計時的字體粗細 | `400` |
| \--nutui-countdown-width | 倒計時的時間區域寬度 | `20px` |
| \--nutui-countdown-height | 倒計時的時間區域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒計時的時間區域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒計時的時間區域margin | `0 2px` |
| \countdown-number-border-radius | 倒計時的時間區域圓角大小 | `2px` |
| \--nutui-countdown-color | 倒計時的時間區域的文字顔色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
| \--nutui-countdown-primary-color | type爲`primary`時,倒計時的時間區域的文字顔色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type爲`primary`時,倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type爲`primary`時,倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请注意,CSS变量名应以双破折号(--)开头。在第145行,CSS变量名缺少了这个前缀。

- | \countdown-number-border-radius | 倒計時的時間區域圓角大小 | `2px` |
+ | \--countdown-number-border-radius | 倒計時的時間區域圓角大小 | `2px` |
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| \--nutui-countdown-color | 倒計時的文字顔色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒計時的字體大小 | `10px` |
| \--nutui-countdown-font-weight | 倒計時的字體粗細 | `400` |
| \--nutui-countdown-width | 倒計時的時間區域寬度 | `20px` |
| \--nutui-countdown-height | 倒計時的時間區域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒計時的時間區域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒計時的時間區域margin | `0 2px` |
| \countdown-number-border-radius | 倒計時的時間區域圓角大小 | `2px` |
| \--nutui-countdown-color | 倒計時的時間區域的文字顔色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
| \--nutui-countdown-primary-color | type爲`primary`時,倒計時的時間區域的文字顔色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type爲`primary`時,倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type爲`primary`時,倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
| \--nutui-countdown-color | 倒計時的文字顔色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒計時的字體大小 | `10px` |
| \--nutui-countdown-font-weight | 倒計時的字體粗細 | `400` |
| \--nutui-countdown-width | 倒計時的時間區域寬度 | `20px` |
| \--nutui-countdown-height | 倒計時的時間區域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒計時的時間區域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒計時的時間區域margin | `0 2px` |
| \--countdown-number-border-radius | 倒計時的時間區域圓角大小 | `2px` |
| \--nutui-countdown-color | 倒計時的時間區域的文字顔色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
| \--nutui-countdown-primary-color | type爲`primary`時,倒計時的時間區域的文字顔色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type爲`primary`時,倒計時的時間區域的背景顔色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type爲`primary`時,倒計時的時間區域的邊框顔色顔色 | `$color-primary` |
Tools
LanguageTool

[locale-violation] ~146-~146: 'light' é un xenismo. É preferíbel dicir "leve" ou "de baixo valor calórico" (GL_BARBARISM_REPLACE)
Context: ...nutui-countdown-color | 倒計時的時間區域的文字顔色 | $color-primary-light | | --nutui-countdown-background-colo...


[locale-violation] ~147-~147: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...r-primary-light| | \--nutui-countdown-background-color | 倒計時的時間區域的背景顔色 |$color-primary...


[locale-violation] ~150-~150: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...#FFFFFF | | \--nutui-countdown-primary-background-color | type爲primary`時,倒計時的時間區域的背景顔色 |...

Markdownlint

151-151: null (MD047, single-trailing-newline)
Files should end with a single newline character

Comment on lines 138 to 151
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请注意,CSS变量名应以双破折号(--)开头。在第145行,CSS变量名缺少了这个前缀。

- | \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
+ | \--countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \--countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
Tools
LanguageTool

[locale-violation] ~146-~146: 'light' é un xenismo. É preferíbel dicir "leve" ou "de baixo valor calórico" (GL_BARBARISM_REPLACE)
Context: ...nutui-countdown-color | 倒计时的时间区域的文字颜色 | $color-primary-light | | --nutui-countdown-background-colo...


[locale-violation] ~147-~147: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...r-primary-light| | \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 |$color-primary...


[locale-violation] ~150-~150: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...#FFFFFF | | \--nutui-countdown-primary-background-color | type为primary`时,倒计时的时间区域的背景颜色 |...

Markdownlint

151-151: null (MD047, single-trailing-newline)
Files should end with a single newline character

Comment on lines 139 to 152
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请注意,CSS变量名应以双破折号(--)开头。在第146行,CSS变量名缺少了这个前缀。

- | \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
+ | \--countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-color | 倒计时的文字颜色 | `$color-primary` |
| \--nutui-countdown-font-size | 倒计时的字体大小 | `10px` |
| \--nutui-countdown-font-weight | 倒计时的字体粗细 | `400` |
| \--nutui-countdown-width | 倒计时的时间区域宽度 | `20px` |
| \--nutui-countdown-height | 倒计时的时间区域高度 | `14px` |
| \--nutui-countdown-number-padding | 倒计时的时间区域padding | `0 1px` |
| \--nutui-countdown-number-margin | 倒计时的时间区域margin | `0 2px` |
| \--countdown-number-border-radius | 倒计时的时间区域圆角大小 | `2px` |
| \--nutui-countdown-color | 倒计时的时间区域的文字颜色 | `$color-primary-light` |
| \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-border-color | 倒计时的时间区域的边框颜色颜色 | `$color-primary` |
| \--nutui-countdown-primary-color | type为`primary`时,倒计时的时间区域的文字颜色 | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | type为`primary`时,倒计时的时间区域的背景颜色 | `$color-primary` |
| \--nutui-countdown-primary-border-color | type为`primary`时,倒计时的时间区域的边框颜色颜色 | `$color-primary` |
Tools
LanguageTool

[locale-violation] ~147-~147: 'light' é un xenismo. É preferíbel dicir "leve" ou "de baixo valor calórico" (GL_BARBARISM_REPLACE)
Context: ...nutui-countdown-color | 倒计时的时间区域的文字颜色 | $color-primary-light | | --nutui-countdown-background-colo...


[locale-violation] ~148-~148: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...r-primary-light| | \--nutui-countdown-background-color | 倒计时的时间区域的背景颜色 |$color-primary...


[locale-violation] ~151-~151: 'background' é un xenismo. É preferíbel dicir "formación de base" (GL_BARBARISM_REPLACE)
Context: ...#FFFFFF | | \--nutui-countdown-primary-background-color | type为primary`时,倒计时的时间区域的背景颜色 |...

Markdownlint

152-152: null (MD047, single-trailing-newline)
Files should end with a single newline character

Comment on lines 136 to 150
| \--nutui-countdown-display | 倒计时的布局方式 | `flex` |
| \--nutui-countdown-color | Countdown text color | `$color-primary` |
| \--nutui-countdown-font-size | Countdown font size | `10px` |
| \--nutui-countdown-font-weight | Countdown font weight | `400` |
| \--nutui-countdown-width | Time area width of the countdown | `20px` |
| \--nutui-countdown-height | Time area height of the countdown | `14px` |
| \--nutui-countdown-number-padding | Countdown time area padding | `0 1px` |
| \--nutui-countdown-number-margin | Countdown time area margin | `0 2px` |
| \countdown-number-border-radius | Countdown time area border radius | `2px` |
| \--nutui-countdown-color | Countdown time area text color | `$color-primary-light` |
| \--nutui-countdown-background-color | Countdown time area background color | `$color-primary` |
| \--nutui-countdown-border-color | Countdown time area border color color | `$color-primary` |
| \--nutui-countdown-primary-color | When type is primary, the text color of the countdown time area | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | When type is primary, the background color of the countdown time area | `$color-primary` |
| \--nutui-countdown-primary-border-color | When type is primary, the border color of the countdown time area is the color | `$color-primary` |
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please ensure all CSS variable names start with two hyphens (--). On line 144, the CSS variable name is missing this prefix.

- | \countdown-number-border-radius | Countdown time area border radius | `2px` |
+ | \--countdown-number-border-radius | Countdown time area border radius | `2px` |
Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| \--nutui-countdown-display | 倒计时的布局方式 | `flex` |
| \--nutui-countdown-color | Countdown text color | `$color-primary` |
| \--nutui-countdown-font-size | Countdown font size | `10px` |
| \--nutui-countdown-font-weight | Countdown font weight | `400` |
| \--nutui-countdown-width | Time area width of the countdown | `20px` |
| \--nutui-countdown-height | Time area height of the countdown | `14px` |
| \--nutui-countdown-number-padding | Countdown time area padding | `0 1px` |
| \--nutui-countdown-number-margin | Countdown time area margin | `0 2px` |
| \countdown-number-border-radius | Countdown time area border radius | `2px` |
| \--nutui-countdown-color | Countdown time area text color | `$color-primary-light` |
| \--nutui-countdown-background-color | Countdown time area background color | `$color-primary` |
| \--nutui-countdown-border-color | Countdown time area border color color | `$color-primary` |
| \--nutui-countdown-primary-color | When type is primary, the text color of the countdown time area | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | When type is primary, the background color of the countdown time area | `$color-primary` |
| \--nutui-countdown-primary-border-color | When type is primary, the border color of the countdown time area is the color | `$color-primary` |
| \--nutui-countdown-display | 倒计时的布局方式 | `flex` |
| \--nutui-countdown-color | Countdown text color | `$color-primary` |
| \--nutui-countdown-font-size | Countdown font size | `10px` |
| \--nutui-countdown-font-weight | Countdown font weight | `400` |
| \--nutui-countdown-width | Time area width of the countdown | `20px` |
| \--nutui-countdown-height | Time area height of the countdown | `14px` |
| \--nutui-countdown-number-padding | Countdown time area padding | `0 1px` |
| \--nutui-countdown-number-margin | Countdown time area margin | `0 2px` |
| \--countdown-number-border-radius | Countdown time area border radius | `2px` |
| \--nutui-countdown-color | Countdown time area text color | `$color-primary-light` |
| \--nutui-countdown-background-color | Countdown time area background color | `$color-primary` |
| \--nutui-countdown-border-color | Countdown time area border color color | `$color-primary` |
| \--nutui-countdown-primary-color | When type is primary, the text color of the countdown time area | `#FFFFFF` |
| \--nutui-countdown-primary-background-color | When type is primary, the background color of the countdown time area | `$color-primary` |
| \--nutui-countdown-primary-border-color | When type is primary, the border color of the countdown time area is the color | `$color-primary` |
Tools
LanguageTool

[duplication] ~147-~147: Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Context: ...rder-color | Countdown time area border color color | $color-primary | | --nutui-countdo...

Markdownlint

150-150: null (MD047, single-trailing-newline)
Files should end with a single newline character

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants