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(select): 增加JSX方式添加Option #562

Merged
merged 3 commits into from
Dec 4, 2020

Conversation

berber1016
Copy link
Contributor

affects: @gio-design/components, website

ISSUES CLOSED: #523, #522, #495, #448

@gio-design/components
Select
Numeric value, selector does not display the selected value
new feature clear
Select supports JSX to pass in Option

List
Option supports render method

@vercel
Copy link

vercel bot commented Dec 1, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/growingio/gio-design/1he2nlpdj
✅ Preview: https://gio-design-git-select.growingio.vercel.app

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #562 (41f4095) into master (aa386e8) will increase coverage by 0.91%.
The diff coverage is 92.30%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #562      +/-   ##
==========================================
+ Coverage   78.61%   79.52%   +0.91%     
==========================================
  Files         124      126       +2     
  Lines        4041     4084      +43     
  Branches     1062     1072      +10     
==========================================
+ Hits         3177     3248      +71     
+ Misses        857      830      -27     
+ Partials        7        6       -1     
Impacted Files Coverage Δ
...ckages/components/src/components/select/Select.tsx 94.76% <90.00%> (-1.87%) ⬇️
packages/components/src/components/list/list.tsx 92.70% <100.00%> (+0.40%) ⬆️
...ages/components/src/components/select/OptGroup.tsx 100.00% <100.00%> (ø)
...kages/components/src/components/select/Options.tsx 100.00% <100.00%> (ø)
...nts/src/components/date-picker/dateRangePicker.tsx 59.21% <0.00%> (+43.42%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e51666c...31927e6. Read the comment docs.

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@azure-pipelines
Copy link

Deploy preview for ready! ✅ https://deploy-preview-562--gio-design.surge.sh

@@ -2,7 +2,7 @@

.@{select-prefix-cls}-option {
max-width: 100%;
height: 40px;
// height: 40px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

不需要直接删除

@@ -1,5 +1,11 @@
import Select from './Select';
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里需要和 #564 同步一下

@azure-pipelines
Copy link

Deploy preview for ready! ✅ https://deploy-preview-562--gio-design.surge.sh

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

@azure-pipelines
Copy link

Deploy preview for gio-design failed.

affects: @gio-design/components, website

ISSUES CLOSED: growingio#523, growingio#522, growingio#495, growingio#448
@azure-pipelines
Copy link

Deploy preview for ready! ✅ https://deploy-preview-562--gio-design.surge.sh

@azure-pipelines
Copy link

Deploy preview for ready! ✅ https://deploy-preview-562--gio-design.surge.sh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants