-
Notifications
You must be signed in to change notification settings - Fork 10
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
日本語の部分を英語にする vol1 #251
日本語の部分を英語にする vol1 #251
Conversation
}) => ( | ||
// MEMO: Add className props to apply style | ||
<a href={href} className={className} {...rest}> | ||
<a href={href} className={className}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{...rest}
を加えると意図しないpropsも入り、警告が出るため変更。
return React.cloneElement(child, childNeighborDisabledProps); | ||
} | ||
return React.cloneElement(child, childProps); | ||
const Button = React.cloneElement(child, { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このあたり変数を減らした
|
||
// MEMO: おそらくstyled-component@5.x.xで型エラーが起こっているので、明示的に宣言している |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
直してみたら型エラーが出なかったので対応。
const margin = getMargin({ | ||
theme, | ||
m: 1, | ||
describe("margin", () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
margin
について、padding
についてそれぞれグルーピングした(出力がみやすくなる
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
ref: #152
ひとまずコンポーネントの修正が不要となる部分は当PRでなおす。
が、propsの初期値は英語に切り替わるものもある。
propsの初期値が変わるコンポーネント
<FileUploader />
<ConfirmModal />
別PRで対応するコンポーネント
<DataTable />
<DatePicker />
<DateRangePicker />
<NavigationRail />
(Footer)<TextField />