[FEAT] 테스트 결과 이미지 저장 기능 구현 #131
Merged
Conversation
This file contains 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
|
@jhsung23 지현님~ 로딩 UI와 SVG로 변경했습니다. 리뷰부탁드려요~ |
jhsung23
approved these changes
Feb 20, 2024
| <Icon icon="caretUp" color="gray-300" size={12} /> | ||
| <div ref={imageRef}>{resultTypeMap[resultTypeId]}</div> | ||
| <div className="flex items-center gap-5xs py-xs"> | ||
| <Button variant="empty" onClick={handleDownloadImage} className="h-[30px]"> |
There was a problem hiding this comment.
button에 isLoading 프롭이 있긴해요! 필요하심 사용하셔도 되고 아니면 지금 유지하셔도 됩니닷
There was a problem hiding this comment.
디자인이 변경돼서 로딩하지 않기로 이야기했어요! disable 처리하려고 하는데 isLoading props 사용해서 구현할게요~
Comment on lines
45
to
51
| // HELP: 더 좋은 방법이 있을까요!? | ||
| // svg가 tsx 파일에서 가능하여 constants로 뺴지 않았습니다. | ||
| const resultTypeMap = { | ||
| 1: <FirstResultType />, | ||
| 2: <SecondResultType />, | ||
| 3: <ThirdResultType />, | ||
| 4: <ForthResultType />, |
There was a problem hiding this comment.
음.. 괜찮다고 생각해요!
다만 결과에 대한 건 모두 서버에서 받아오는게 더 낫다고 생각하긴 하는데, 프론트에 저장하고 사용하시려는 이유가 있을까요?? (프백간 합의만 된다면 상관없긴 하지만요~)
There was a problem hiding this comment.
아직 희건님 오시기 전이라 미리 테스트 해놓은거였어요~~ 백엔드 svg도 내려줄 수 있을까라는 생각이 들었어요!
| import { Range } from '@/types/util'; | ||
|
|
||
| // TODO : 백엔드와 논의하여 resultTypeId 프로퍼티 결정 | ||
| const ImageBox = ({ resultTypeId = 4 }: { resultTypeId: Range<1, 5> }) => { |
There was a problem hiding this comment.
여기 타입 부분만 resultTypeId에서 가져오도록 수정하면 될 것 같아요.
타입 아이디가 하나 더 늘면 객체에도 추가해야하고 Range 범위도 변경해야 하니까 keyof typeof resultTypeId를 넣어주는 것도 좋아보여요.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
✨ 작업 내용
close #18
📚 작업 결과
웹
2024-02-19.7.48.35.mov
모바일
default.MP4
🙏 기타 참고 사항
디자인팀과 논의해야할 부분이 많이 보입니다... 로딩중일 경우 어떻게 표현할 것인지?! 확장자는 Png라고 했으나 화질 문제가 있어보입니다.
@jhsung23 디자인 변경된 것도 반영하여 Merge하겠습니다.
✅ PR 등록 전 확인 후 체크해 주세요! (x 표시 해 주세요.)