refactor: 각종 리팩터링 모음 #391
Merged
refactor: 각종 리팩터링 모음 #391
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
Closed
4 tasks
WaiNaat
requested changes
Sep 22, 2023
| const Select = ({ value, options, onChange, placeholder }: SelectProps) => { | ||
| const { isOn: isOpen, off: close, toggle } = useToggle(); | ||
|
|
||
| const [optionHeight, setOptionHeight] = useState(DEFAULT_HEIGHT); |
There was a problem hiding this comment.
C
굳이 새로운 상태로 useEffect 안에서 처리하지 않아도 구현이 가능할 것 같아요
Suggested change
| const [optionHeight, setOptionHeight] = useState(DEFAULT_HEIGHT); | |
| const currentPos = selectRef.current?.getBoundingClientRect(); | |
| const optionHeight = options.length * 40; | |
| const topPosition = | |
| currentPos && currentPos.top + optionHeight + DEFAULT_HEIGHT > window.innerHeight | |
| ? -optionHeight | |
| : DEFAULT_HEIGHT; |
Comment on lines
+16
to
+22
| useEffect(() => { | ||
| if (!isOpen) { | ||
| document.body.style.position = 'relative'; | ||
| return; | ||
| } | ||
| document.body.style.position = 'fixed'; | ||
| }, [isOpen]); |
There was a problem hiding this comment.
모바일에서 모달이 열리면 뒤에 스크롤이 생기더라구요! 그래서 추가했습니다
rawfishthelgh
pushed a commit
that referenced
this pull request
Sep 22, 2023
* refactor: 사전 식물 이미지 해상도를 조정한다 (#387) * feat: 사진 크기에 따라 url을 제시하는 유틸 함수 * feat: 확장자별로 사진 로딩 시도 하나의 url로 로딩 실패하면 다음 url을 불러오는 방식 * refactor: Image의 size 픽셀만 가능하게 변경 유틸 함수가 계산 이슈로 픽셀만 받을 수 있어서 그에 맞게 변경함 * test: 원인모를 cypress 실패 주석처리 로컬에서는 너무나 잘 되지만 github action에서는 임의의 확률로 실패하는 코드 * test: 동작이 이상한 테스트 코드 제거 github action에서만 실패하는 수상한 코드 임시 주석 처리 * chore: cypress github action 버전 업그레이드 * chore: github action 수정 실험중입니다 * test: auth redirection 토스트 확인 삭제 * chore: e2e ci 실패시 스크린샷 저장 * chore: upload artifact 버전 명시 * chore: upload artifact 삭제 동작 방식에 대한 빠른 이해 실패 * test: 접속 후 대기시간 추가 리다이렉션을 확실히 하도록 하기 위함 * test: github actions에서만 안되는 테스트 제거 * chore: e2e ci 실패 슬랙 알림 활성화 * fix: 사용자 사진 업로드 시 미리보기가 안 바뀌는 문제 (#390) * feat: url이 피움 static인지 확인하는 유틸 함수 * fix: src를 바꿔도 이미지가 변하지 않는 현상 - 맨 처음 주어진 src로 ref 안에 배열을 가두어 새로운 src가 주어져도 사진이 변하지 않음 - setErrorImage 자체는 일종의 비제어 컴포넌트처럼 동작, ref를 이용할 이유 X * refactor: useFileUpload 호출마다 새 url 생성 방지 * test: msw mock data image url 변경 피움 static 서버에서 받아오도록 수정 * refactor: 각종 리팩터링 모음 (#391) * design: line-height 설정 * refactor: 모두의 정원 margin-bottom 설정 * refactor: 위치에 따라 select박스 위치 변경 * design: line-height 설정 * refactor: pending일 경우에 button block * refactor: modal 오픈시에 scroll 방지 * refactor: select 상태 제거 * design: 모두의 정원 header 수정 * chore: v1.1.1로 업데이트 --------- Co-authored-by: 정호진 <[email protected]>
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.
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
거의 모든 font에 line-height를 추가했습니다. line-height 기준은 현재 폰트 크기의 1.2배 정도입니다!
셀렉트 박스는 위치는 셀렉트 박스가 펼쳐졌을 때 초과되지 않을 때 길이를 기준으로 초과하지 않는 방향으로 가는 쪽으로 했습니다.
달력 컴포넌트에 날짜 변환하는게 쉽지 않네요... 뭔가 아래 처럼 해보고 싶었는데... 이건 다음 기회에 해봐야 할 것 같습니다...
