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

feat: support input number component #2630

Merged
merged 4 commits into from
Jun 6, 2023

Conversation

Ricbet
Copy link
Member

@Ricbet Ricbet commented Apr 21, 2023

Types

  • 🎉 New Features

Background or solution

🤖 Generated by Copilot at faac3f4

  • Add InputNumber component to the components package (link, link, link)
  • This component is a wrapper around the rc-input-number dependency, which is added to the package.json file (link)
  • Modify the base.less file to hide the default up and down buttons for the number input in different browsers (link, link)

基于 rc-input-number 实现
dark 主题
Kapture 2023-04-21 at 15 27 41

light 主题
Kapture 2023-04-21 at 15 26 06

Changelog

🤖 Generated by Copilot at faac3f4

This pull request adds a new InputNumber component to the components package, which allows users to input numbers with up and down buttons. The component uses the rc-input-number dependency and customizes its styles and props. The component also updates the global styles for the number input and exports it from the input folder.

@Ricbet Ricbet self-assigned this Apr 21, 2023
@opensumi opensumi bot added the 🎨 feature feature required label Apr 21, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 21, 2023

ChatGPT Code Review:

  1. 代码没有错误且没有拼写错误。
  2. 代码提交是向 components 包中添加了一个名为 InputNumber 的组件。这个组件封装了 rc-input-number的依赖模块。它提供了一个方便的方法来输入数字。这个组件还修改了 base.less 这个文件的样式来隐藏浏览器中默认的上下按钮,以提高用户体验。组件提供了两种不同的主题(dark 和 light)的显示效果。

@erha19
Copy link
Member

erha19 commented May 4, 2023

这里 +/- 号的样式 disable 样式有点不统一,+ 点到底左下边框都没了,-点到底只有左边框没了 @Ricbet

@Ricbet Ricbet force-pushed the feat/support-input-number-component branch from faac3f4 to dd25b1a Compare May 12, 2023 03:13
@Ricbet Ricbet force-pushed the feat/support-input-number-component branch from 5c1bec6 to c191850 Compare May 19, 2023 09:33
@erha19
Copy link
Member

erha19 commented May 24, 2023

@Ricbet CI 处理一下

@Ricbet Ricbet force-pushed the feat/support-input-number-component branch from c191850 to 665368d Compare June 5, 2023 09:00
@codecov
Copy link

codecov bot commented Jun 5, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (319484c) 57.75% compared to head (665368d) 57.75%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2630   +/-   ##
=======================================
  Coverage   57.75%   57.75%           
=======================================
  Files        1330     1330           
  Lines       83676    83676           
  Branches    17396    17396           
=======================================
+ Hits        48326    48329    +3     
+ Misses      32128    32126    -2     
+ Partials     3222     3221    -1     
Flag Coverage Δ
jsdom 52.76% <ø> (+<0.01%) ⬆️
node 16.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@erha19 erha19 merged commit a7d5605 into main Jun 6, 2023
12 checks passed
@erha19 erha19 deleted the feat/support-input-number-component branch June 6, 2023 02:03
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.

None yet

2 participants