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

BREAKING CHANGE: Update components #319

Merged
merged 233 commits into from
Nov 1, 2019
Merged

BREAKING CHANGE: Update components #319

merged 233 commits into from
Nov 1, 2019

Conversation

im36-123
Copy link
Contributor

@im36-123 im36-123 commented Sep 10, 2019

im36-123 and others added 30 commits August 6, 2019 15:36
BREAKING CHANGE: remove size of tasting and trenta
* chore: yarn add react-icons

* feat: add some icons from react-icons/fa

* refactor: change props of original icon

BREAKING CHANGE: original icon gets the same props as react-icons

* chore: update snapshot

* chore: export Icon component

* fix: remove 'Base', 'Icon' from IGNORE_DIRS
* chore: change key name of color to constant case

BREAKING CHANGE: pascal case doesn't support.

* chore: update snapshots
# Conflicts:
#	src/components/CheckboxLabel/CheckboxLabel.tsx
#	src/components/Field/Field.tsx
#	src/components/Input/Input.tsx
#	src/components/Pagination/PaginationItem.tsx
#	src/components/RadioLabel/RadioLabel.tsx
nabeliwo and others added 23 commits October 30, 2019 18:45
Add Dialog component and remove Modal component
Fix dropdown content position bug
Change interface of Button, Select and Checkbox to conform to the DOM interface
Change Input interface to conform to the DOM interface
@nabeliwo nabeliwo marked this pull request as ready for review November 1, 2019 04:59
@nabeliwo nabeliwo requested a review from a team as a code owner November 1, 2019 04:59
@ghost ghost requested review from nabeliwo and removed request for a team November 1, 2019 05:00
@nabeliwo nabeliwo changed the title [WIP] Features/ttmz BREAKING CHANGE: Update components Nov 1, 2019
@nabeliwo nabeliwo merged commit e5ce046 into master Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants