-
Notifications
You must be signed in to change notification settings - Fork 13
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
[3주차] 전시원 미션 제출합니다. #23
base: master
Are you sure you want to change the base?
Changes from all commits
c6feae2
f5b938e
4427d83
4b54ec9
e8bbcba
6b36e58
28017c7
bb2753b
e9d358b
743fdc4
352336c
cd6291c
11967de
5274f66
544079f
b2cebe8
ac130c1
8438aa6
ac54062
e8a6881
0ad9ee6
ddcb16c
c3ef0e8
fe2b790
9b333ab
85b97cd
7554c72
f0f19a2
96519d5
1976c20
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
import React from 'react'; | ||
|
||
export interface ITodoList { | ||
contents: string; | ||
id: number; | ||
isDone: boolean; | ||
} | ||
export interface IYetListProps { | ||
list: []; | ||
yetNum: number; | ||
onToggle: (e: React.MouseEvent<HTMLButtonElement>) => void; | ||
onDelete: (e: React.MouseEvent<HTMLButtonElement>) => void; | ||
} | ||
|
||
export interface IInputFormProps { | ||
handleFormSubmit: (e: React.SyntheticEvent) => void; | ||
handleContentsChange: (e: React.ChangeEvent<HTMLInputElement>) => void; | ||
contents: string; | ||
} | ||
|
||
export interface IDoneList { | ||
list: []; | ||
onToggle: (e: React.MouseEvent<HTMLButtonElement>) => void; | ||
doneNum: number; | ||
} | ||
// styled-component에 들어가는 property 에 대한 정의 | ||
export interface IListBtn { | ||
onClick: (e: React.MouseEvent<HTMLButtonElement>) => void; | ||
} | ||
Comment on lines
+3
to
+29
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Props까지 모두 interface로 정리하신게 인상깊은 것 같아요. Type이 아닌 Interface로 정의하신 점이 궁금하게 되는 것 같습니다, 찾아보니까 props는 interface로 정의하는게 관례라고 하는군요. 감사합니다! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 오,, 저는 이번에 타입스크립트 처음 써서 그런 관례를 알고 쓴 건 아니었는데 찾아서 정리해주셔서 감사합니다 |
||
|
||
// 클릭 : MouseEvent | ||
// 입력 : ChangeEvent | ||
// 제출 : SyntheticEvent |
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.
어떤 array인지 좀더 정확하게...!