[Feat] 인수인계 노트 페이지 수정 #329
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
Bowoon1216
approved these changes
Nov 19, 2024
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.
해당 이슈 번호
closed #327
체크리스트
💎 PR Point
전체적으로 자잘한 스타일이나 플로우 ? 수정입니다 !
gui보니까 인수인계 노트 페이지 일 때도 RouteNav가 인수인계 노트 메뉴로 되어있어서 pathname을 추가했습니다.
라디오 버튼을 원래 라디오버튼이 디폴트로 제공해주는 디자인에서 선택됐을 때 색상만 변경했었는데, 이를 수정해서
커스텀하는 방식으로 다시 작성했습니다.인수인계 노트의 템플릿, 자유 작성 섹션이 화면을 축소했을 때 잘린 상태에서 스크롤 되는 문제가 있었는데 뷰 전체 height 기준으로 계산하는 방식으로 수정했습니다.
인수인계 노트에서 디폴트 값이 체크가 안되어있더라고요 ? 알고보니 RadioButton에서
checkedprop을 넘겨주지 않아서 생긴 바보 이슈였습니다.인수인계 노트의
태그부분들도 더미로 렌더링 했습니다. 색상 부분은 서버에서 넘겨줄지 랜덤 값으로 지정하는지 아직 모르겠어서 일단 지정해주지 않았습니다.이전 스프린트에서 로그인시
쇼케이스로 이동했으나, 이를 모두대시보드로 이동하는 걸로 수정했습니다!📌스크린샷 (선택)