-
Notifications
You must be signed in to change notification settings - Fork 0
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
テーブル機能を実装 #6
テーブル機能を実装 #6
Conversation
フィルタすべき項目は以上ですか?本当に?? |
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.
全体的に甘いです
pages/users.tsx
Outdated
return ( | ||
<div style={{ width: "100%" }} className="overflow-auto"> | ||
{allTableDataTypes.map(t => ( | ||
<div style={{ width: "90vw", margin: "auto" }} className="overflow-auto"> |
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.
左右に余白を作りたいのであれば、そもそもbootstrapのcontainerなりをしっかり利用してください。使い方などはDBのページを参考にしてください。
pages/users.tsx
Outdated
striped | ||
bordered | ||
hover | ||
style={{ tableLayout: "fixed", overflow: "scroll" }} |
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.
overflowはscrollだとX,Yどちらもスクロールが出てしまいます。autoにして必要な部分だけ表示されるようにしてください
pages/users.tsx
Outdated
<> | ||
<input type="checkbox" key={t} onChange={(e) => { onCheckFilter(e.target.checked, t) }} /> | ||
<input |
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.
このまま標準の使いにくいinput使うんですか?Bootstrapのcheckboxを利用してください。
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.
まだ実際に動作確認はできていませんが、とりあえずコードで気になるところだけ
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.
リリースも遅れていて早めにリリースしたいとのことなのでこちらで修正してしまいました。最後にそちらで一度動作確認を取った後にMergeしちゃってください
No description provided.