feat: 구인 글 메인 페이지 & 구인 글 상세페이지 & 산출물 미리보기 페이지 구현 #43
Conversation
prgmr99
changed the title
src/components/meeteam/requiredInformation/RequiredInformation.styled.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
승준님 전에 말씀드렸던 멤버 삭제 하고 나서 삭제 했던 멤버까지 추가되는 버그가 다시 발생하는 듯 합니다..!🤔
There was a problem hiding this comment.
넵!!! 제가 열심히 고쳐보겠습니다!!!
저 모르게 에러 찾으려고 노력해주셔서 감사해요ㅠㅠ
| const onChangeImg = (event: React.ChangeEvent<HTMLInputElement>) => { | ||
| if (event.target.files !== null) { | ||
| setImgFileName(event.target.files[0].name); | ||
| const selectedFiles = event.target.files as FileList; | ||
| setImgFile(URL.createObjectURL(selectedFiles?.[0])); | ||
| const selectedFiles = event.target.files[0]; | ||
| const reader = new FileReader(); | ||
| reader.readAsDataURL(selectedFiles); | ||
|
|
||
| return new Promise<void>(resolve => { | ||
| reader.onload = () => { | ||
| setImgFile(reader.result as any); | ||
| resolve(); | ||
| }; | ||
| }); | ||
| } | ||
| }, []); | ||
| }; |
There was a problem hiding this comment.
혹시 해당 코드에 대해서 설명해주실 수 있으실까요?🤔
There was a problem hiding this comment.
위 코드는 이미지 파일이 변경될때마다 저장하는 과정을 담은 코드입니다.
제가 이전에 참고한 자료는 아래의 블로그입니다!!!
https://nukw0n-dev.tistory.com/30
그런데 더 알아보니 위의 코드에서 나온 FileReader를 사용하는 것보다 성능상 더 좋은 방식이 있어서 수정하여 반영했습니다!
https://mieumje.tistory.com/164
|
|
||
| const handleNextSlide = () => { | ||
| setIsBack(false); | ||
| setSlideIndex(prevIndex => (prevIndex === lists.length - 1 ? 0 : prevIndex + 1)); |
There was a problem hiding this comment.
매직 넘버를 상수화 시키면 좋을 것 같습니당:-)
There was a problem hiding this comment.
혹시 어떤게 매직 넘버일까요...? 매직 넘버가 없지 않나요??!?!
There was a problem hiding this comment.
저는 인덱스 0도 매직 넘버라고 생각햇는데 아닌가용..?
There was a problem hiding this comment.
제가 아는 매직 넘버는 의미있는 상수를 숫자로 지정하는 것으로 알고 있는데,
인덱스 0은 제가 숫자로 지정한게 아니어서 매직 넘버가 아니지 않나요...?!?!
배열의 인덱스까지 매직넘버는 아닌 것 같은데 혹시 제가 모르는 참고하신 자료가 있을까요??ㅠㅠ
위의 코드는 슬라이드 배열을 다루는 로직이어서 0을 따로 변경해주지 않아도 알 수 있지 않을까요??
저의 개인적인 의견입니다!!
There was a problem hiding this comment.
아 그렇군요..! 저는 그냥 단지 의견이긴 했습니다..!! 저는 항상 인덱스 상수 0도 STRAT_INDEX 이렇게 바꿨었거든요...!
제가 잘못 알고 있었나 봅니다..ㅎㅎ
| } | ||
|
|
||
| .container-info__tags { | ||
| margin-top: 1.73rem; |
There was a problem hiding this comment.
해당 부분 css 를 주신 이유가 궁금해요! 실제로 좀 제목과 필수정보 사이의 간격이 많이 큰 것 같아서요!🤔
There was a problem hiding this comment.
디자인이 안 된 페이지라 제가 임의로 수정하고 있었습니다!!
There was a problem hiding this comment.
별로는 아닌데 저는 간격이 조금 크게 보이는 것 같아용..!! 다른 간격에 비해서요..!!
There was a problem hiding this comment.
아하! 알겠습니다!
이슈 #49 을 작업하면서 디자인을 전반적으로 다듬을 계획이어서 그때 다시 수정해놓겠습니다!!
| } | ||
|
|
||
| .container-info__introduction { | ||
| margin-top: 4.05rem; |
There was a problem hiding this comment.
해당 css도 간격이 피그마 상보다 커서 문의드립니당!
There was a problem hiding this comment.
이것도 디자인이 안 된 페이지라 제가 임의로 수정하고 있었습니다!!
There was a problem hiding this comment.
이것도 마찬가지로 간격이 조금 크게 보이는 것 같아용..!!
| <div className='container-images__title'> | ||
| <span>[커뮤니티 웹 서비스 프로젝트]</span> | ||
| <Tag type={'완료'} /> | ||
| </div> |
There was a problem hiding this comment.
Title 컴포넌트에 Tag 컴포넌트도 적용해서 나중에 공통 컴포넌트로 대체해서 적용하면 좋을 것 같습니다!!
| </AnimatePresence> | ||
| </div> | ||
| <div className='container-info'> | ||
| <span className='container-info__subtitle'>[커뮤니티 웹 서비스 프로젝트]</span> |
There was a problem hiding this comment.
위에 제목이 있는데 과연 추가적으로 제목이 필요할까요?🤔
There was a problem hiding this comment.
피그마에는 둘 다 제목으로 되어 있는데 하나는 밋팀 이름이고 하나는 제목이에요!!
그래서 원래는 둘이 다른 내용입니다!
There was a problem hiding this comment.
아하!! 몰랐습니다ㅠㅠ 밋팀 이름이 최상단이겠네요 그럼??!
|
승준님~~ 고생 많으셨습니다!🔥 리뷰 확인 부탁드립니다~! |
There was a problem hiding this comment.
승준님..! 혹시 해당 필터 기능의 드롭다운 모양에 transition css 주신 이유가 궁금해요!! 뭔가 회전을 하는 느낌을 주시려고 하시는 건가요??🤔
There was a problem hiding this comment.
추가적으로
위와 같이 필터에서 의도적으로 글씨를 우측 정렬 하신 건가요??🤔
There was a problem hiding this comment.
승준님..! 혹시 해당 필터 기능의 드롭다운 모양에 transition css 주신 이유가 궁금해요!! 뭔가 회전을 하는 느낌을 주시려고 하시는 건가요??🤔
ㅋㅋㅋ네... 별로인가요?
원티드 따라하고 싶어서...
There was a problem hiding this comment.
추가적으로
위와 같이 필터에서 의도적으로 글씨를 우측 정렬 하신 건가요??🤔
넵..!! 일단 그렇습니다..! 원래 가운데로 정렬하고 싶었고, 화살표만 우측에 나두고 싶었는데
제 마음대로 안 되어서 일단 저렇게 구현했고,, 저도 인지하고 있어요!! 이번주에 수정 사항들 작업할 때 다시 한 번 더 작업해보려구요!
There was a problem hiding this comment.
이상하지 않아요!! 한 0.1s 로 주면 딱 좋을 것 같아요~!!👍🏻
네넵!!👍🏻
구현 내용/방법
리뷰 필요
close #42