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(components): add searchbar and refactor input #153

Merged
merged 66 commits into from
Sep 8, 2020

Conversation

1eeing
Copy link
Contributor

@1eeing 1eeing commented Aug 24, 2020

No description provided.

leeing and others added 30 commits June 11, 2020 20:39
affects: @gio-design/components, website
affects: @gio-design/components
affects: @gio-design/components, website
affects: @gio-design/components
affects: @gio-design/components, website, icon
affects: @gio-design/components
affects: @gio-design/components, @gio-design/icon, @gio-design/themes, @gio-design/tokens, website
affects: website
# Conflicts:
#	packages/website/src/components/basic/toast/index.zh-CN.md
# Conflicts:
#	packages/components/src/index.ts
affects: @gio-design/components, @gio-design/icons, website
affects: @gio-design/components
affects: @gio-design/components
affects: @gio-design/components
# Conflicts:
#	packages/icons/src/index.tsx
leeing added 2 commits August 24, 2020 19:36
affects: @gio-design/components, @gio-design/icons, @gio-design/tokens, website
@azure-pipelines
Copy link

Deploy preview for gio-design failed.

leeing added 2 commits August 25, 2020 12:45
affects: @gio-design/components, @gio-design/icons
@codecov
Copy link

codecov bot commented Aug 25, 2020

Codecov Report

Merging #153 into master will increase coverage by 4.89%.
The diff coverage is 93.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
+ Coverage   65.78%   70.68%   +4.89%     
==========================================
  Files          78       77       -1     
  Lines        2040     1968      -72     
  Branches      542      514      -28     
==========================================
+ Hits         1342     1391      +49     
+ Misses        693      572     -121     
  Partials        5        5              
Impacted Files Coverage Δ
...omponents/src/components/pagination/Pagination.tsx 83.87% <ø> (+0.89%) ⬆️
...components/src/components/input/hooks/useEnter.tsx 85.71% <85.71%> (ø)
...es/components/src/components/input/InputNumber.tsx 86.66% <88.88%> (+70.00%) ⬆️
packages/components/src/components/input/Input.tsx 93.33% <100.00%> (+2.02%) ⬆️
...kages/components/src/components/input/Password.tsx 100.00% <100.00%> (+80.64%) ⬆️
...kages/components/src/components/input/TextArea.tsx 100.00% <100.00%> (+77.77%) ⬆️
...ts/src/components/upload/triggers/InputTrigger.tsx 86.36% <100.00%> (+1.74%) ⬆️
...ponents/src/components/config-provider/context.tsx 100.00% <0.00%> (+65.00%) ⬆️
... and 1 more

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 d7bfc4d...eb821f2. Read the comment docs.

@azure-pipelines
Copy link

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

@gh2049 gh2049 requested review from huskylengcb and jack0pan August 31, 2020 07:56
affects: @gio-design/components, @gio-design/icons, @gio-design/tokens, website
@azure-pipelines
Copy link

Deploy preview for gio-design failed.

affects: @gio-design/components, @gio-design/tokens, website
@azure-pipelines
Copy link

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

@1eeing 1eeing changed the title add searchbar and refactor input feat: add searchbar and refactor input Sep 1, 2020
@1eeing 1eeing changed the title feat: add searchbar and refactor input feat(components): add searchbar and refactor input Sep 1, 2020
packages/components/src/components/input/BaseInput.tsx Outdated Show resolved Hide resolved
packages/components/src/components/input/interfaces.ts Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
packages/components/src/components/input/interfaces.ts Outdated Show resolved Hide resolved

export { InputNumber, InputHidden, Password, TextArea };
export { InputProps } from './types';
export { InputNumber, Password, TextArea };

export default class extends React.Component<InputProps> {
Copy link
Member

Choose a reason for hiding this comment

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

这个定义还是放在 Input 里吧,这样 index.tsx 只用来 import 和 export,这样 index.tsx 不需要单测覆盖。

packages/components/src/components/input/BaseInput.tsx Outdated Show resolved Hide resolved
packages/components/src/components/input/Input.tsx Outdated Show resolved Hide resolved
packages/components/src/components/input/Input.tsx Outdated Show resolved Hide resolved
@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@azure-pipelines
Copy link

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

@1eeing 1eeing merged commit c292e24 into growingio:master Sep 8, 2020
@jack0pan jack0pan added enhancement New feature or request 📎 icon labels Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants