Skip to content

투두리스트 피드백 추가 #6

Description

@DevelopSoo
  1. 파라미터 이름을 item이라 좀 헷갈리네요.

    const handlePlusMinusInput = (item: boolean) => {
    setInput(item);
    };

  2. 실제 실행되는 곳엔 id라고 하지만 파라미터 이름은 index네요. 헷갈리지 않게 수정합시다.

    const handleToggleCheck = (index: number) => {

    const handleDelete = (index: number) => {

  3. 이거 수정안됐습니다. filter로 걸렀으니 삼항연산자 쓸 필요 없습니다.

    {el.check ? (
    <CheckedTextList>{el.text}</CheckedTextList>
    ) : (
    <TextList>{el.text}</TextList>
    )}

    {el.check ? (
    <CheckedTextList>{el.text}</CheckedTextList>
    ) : (
    <TextList key={el.id}>{el.text}</TextList>
    )}

  4. event 객체만 들어가면 위에서 선언하고 그냥 함수 이름만 넣어도 됩니다.
    onChange={handleText}
    onKeyDown={activeEnter}

    onChange={(e) => handleText(e)}
    onKeyDown={(e) => activeEnter(e)}

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions