Conversation
hyesngy
approved these changes
Jun 4, 2025
Member
hyesngy
left a comment
hyesngy
left a comment
There was a problem hiding this comment.
10주차 워크북까지 정말 긴 여정이었습니다.
그동안 모두 고생 많으셨습니다! 👏🏻👏🏻👏🏻 짧지 않은 기간동안 의미 있는 시간이 되었기를 바랍니다.
이제 기본 워크북은 모두 완주하셨지만, 11-12주차 워크북도 도전해보시길 추천드립니다! 11주차에서는 Vercel을 활용한 배포와 CI/CD 구성을 다루고, 12주차에서는 WebSocket을 이용한 실시간 통신과 Cypress를 활용한 E2E 테스트를 다룹니다.
앞으로도 꾸준히 학습하고 성장하시길 응원하며, 데모데이까지 화이팅 입니다! 👍🏻👍🏻👍🏻
Comment on lines
+24
to
+44
| return createPortal( | ||
| <div className="fixed inset-0 bg-black/70 z-50 flex items-center justify-center p-4"> | ||
| <div className="bg-gray-900 text-white max-w-4xl w-full rounded-lg shadow-lg overflow-auto max-h-[90vh] relative"> | ||
| <button | ||
| onClick={() => navigate(-1)} | ||
| className="absolute top-3 right-3 text-white text-2xl" | ||
| > | ||
| × | ||
| </button> | ||
| {isLoading ? ( | ||
| <p className="text-center my-10">영화 정보를 불러오는 중입니다...</p> | ||
| ) : error ? ( | ||
| <p className="text-center my-10 text-red-500">에러가 발생했습니다.</p> | ||
| ) : ( | ||
| <MovieDetailContent movie={movie} /> | ||
| )} | ||
| </div> | ||
| </div>, | ||
| document.body | ||
| ); | ||
| }; |
Member
There was a problem hiding this comment.
미션 1에서 영화 검색 결과로 나온 영화를 클릭하면 모달 형태로 영화의 상세 정보를 불러오도록 해야합니다.
하지만 현재 detail.tsx 에서 createPortal을 사용해 모달처럼 렌더링하고 있지만, 실제로는 useNavigate로 새로운 라우트로 이동하는 방식으로 구현되어 있습니다. 이는 진정한 모달이 아니라 모달 스타일의 페이지 전환이기 때문에, 미션에 맞게 수정이 필요해 보입니다!
Member
There was a problem hiding this comment.
🚀Challenge 미션에 따라 프로젝트 전체의 성능 최적화나 UX 개선에도 도전해보세요! 코드 품질 측면에서도 중복 제거, 책임 분리, 예외 처리 개선 등을 통해 가독성과 유지보수성이 좋은 코드로 리팩토링 해보면 어떨까요 😊
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 미션 번호
10주차 Misson 1
📋 구현 사항
📎 스크린샷
✅ 체크리스트
🤔 질문 사항