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

<Table />を角付きに戻す #134

Merged
merged 3 commits into from
Jul 27, 2020
Merged

<Table />を角付きに戻す #134

merged 3 commits into from
Jul 27, 2020

Conversation

youchann
Copy link
Contributor

@youchann youchann commented Jul 27, 2020

@youchann youchann self-assigned this Jul 27, 2020
@youchann youchann changed the title テーブルを角付きに戻す <Table />を角付きに戻す Jul 27, 2020
@youchann youchann requested a review from noronaoki July 27, 2020 05:21
@youchann
Copy link
Contributor Author

@noronaoki

前の角が丸くない<Table />に戻すだけでOKですかね...?

https://deploy-preview-134--ingred-ui.netlify.app/?path=/story/table--overview

@noronaoki
Copy link
Contributor

@yutaro1031 それで大丈夫です!

src/typings.d.ts Outdated
@@ -17,3 +17,9 @@ declare module "*.svg" {
export default svgUrl;
export { svgComponent as ReactComponent };
}

type DeepPartial<T> = T extends Function
Copy link
Contributor Author

Choose a reason for hiding this comment

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

拾い物のDeepPartial入れてみた

Copy link
Contributor

Choose a reason for hiding this comment

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

拾ったurlとかあるといいかも
reduxだと

export type DeepPartial<T> = {
  [K in keyof T]?: T[K] extends object ? DeepPartial<T[K]> : T[K]
}

こんな感じになってるね

Copy link
Contributor Author

Choose a reason for hiding this comment

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

たしかに...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reduxからのコピペの方が安心感あるので、reduxから取ってきますね!

Copy link
Contributor

@kinokoruumu kinokoruumu left a comment

Choose a reason for hiding this comment

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

LGTMです!

src/typings.d.ts Outdated
@@ -17,3 +17,9 @@ declare module "*.svg" {
export default svgUrl;
export { svgComponent as ReactComponent };
}

type DeepPartial<T> = T extends Function
Copy link
Contributor

Choose a reason for hiding this comment

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

拾ったurlとかあるといいかも
reduxだと

export type DeepPartial<T> = {
  [K in keyof T]?: T[K] extends object ? DeepPartial<T[K]> : T[K]
}

こんな感じになってるね

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.

None yet

3 participants