-
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
[FE] feat: 문제 보여주기 #153
[FE] feat: 문제 보여주기 #153
Conversation
퀴즈데이터를 받아오는 함수 구현 (#151)
공통 Dark버튼 추가
문제를 모달로 입력받고 보여주는 기능추가 변경하기 버튼을 눌러 다시 입력받고, 변경하는 기능 추가 (#151)
QuizViewSection컴포넌트 파일로 분리 (#151)
문제 로드중 로딩 컴포넌트를 보여주도록 처리 (#151)
로딩 컴포넌트 개선 (#151)
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.
LGTM!
@@ -2,12 +2,13 @@ import useFocus from '@/hooks/useFocus'; | |||
import useInput from '@/hooks/useInput'; | |||
import useModal from '@/hooks/useModal'; | |||
|
|||
export default function LinkInputModal() { | |||
export default function LinkInputModal({ setURL }: { setURL: React.Dispatch<React.SetStateAction<string>> }) { |
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.
다른 컴포넌트와 동일하게 Props를 interface로 정의하는것은 어떨까요??
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.
모달 컴포넌트의 경우 useModal의 Props type때문에 타입 정의를 명확하게 할 수가 없네요 ㅠㅠ
PR 설명
문제 보여주기 기능 추가
✅ 완료한 기능 명세
📸 스크린샷
2023-11-28.20-47-53.mp4
고민과 해결과정
overflow-x처리, click disable
iframe내부의 body의 overflow를 해제시켜줘야했다.
fetching이 오래걸리는 문제
isLoading이면 로딩 컴포넌트를 보여주는 형태로 UX를 개선했다.
Suspense는 이후에 도입해볼 생각이다.