Skip to content

[AIBE3/2팀/김정호] TODO APP 만들기 완료#25

Open
Unoguna wants to merge 5 commits intosik2:mainfrom
Unoguna:main
Open

[AIBE3/2팀/김정호] TODO APP 만들기 완료#25
Unoguna wants to merge 5 commits intosik2:mainfrom
Unoguna:main

Conversation

@Unoguna
Copy link

@Unoguna Unoguna commented Jul 9, 2025

📝 React Todo App

Vite + React + Tailwind CSS로 만든Todo 리스트 애플리케이션

📸 스크린샷

image

🚀 주요 기능

  • 할 일 추가
  • 할 일 완료 체크/해제
  • 할 일 삭제
  • 로컬 스토리지 저장
  • Tailwind로 깔끔한 UI

🛠️ 사용 기술

@Bsbd0205
Copy link

수고하셨습니다. 저는 테일윈드 해보려다 포기해서 같은 주제지만 다른 css를 볼 수 있어 좋았습니다.

Copy link

@seung-in-Yoo seung-in-Yoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 전체적으로 되게 깔끔한 코드인거같아요

<meta charset="UTF-8" />
<link rel="icon" type="image/svg+xml" href="/vite.svg" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Vite + React</title>

Choose a reason for hiding this comment

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

사소하지만 제목도 바꾸는게 더 좋다는 생각이 들어요

}

.logo {
height: 6em;

Choose a reason for hiding this comment

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

rem/em 단위 사용하신거 좋습니다!

const handleOnSubmit = (e) => {
e.preventDefault()
const form = e.target
const value = form.todo.value.trim()

Choose a reason for hiding this comment

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

공백 제거 해주신거 좋아요

}

const toggleTodo = (seletedId) => {
const updateTodos = todos.map((todo) => (todo.id == seletedId ? { ...todo, checked: !todo.checked } : todo))

Choose a reason for hiding this comment

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

JS이기 때문에 == 대신 === 사용을 권장드려요!

)
})

const lastId = useRef(todos.length > 0 ? Math.max(...todos.map((todo) => todo.id)) + 1 : 1)

Choose a reason for hiding this comment

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

현재 이 코드 ID 충돌 우려가 있는것 같은데 한번 찾아보셔도 좋을 것 같아요

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.

3 participants