feat(fe): change design in duplicate course and add semester#3499
Conversation
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
💀 Preview Destroyed This PR has been inactive for 28 days. Push a new commit if you want to use preview again. |
sanghyeonjung
force-pushed
the
t2610-duplicate-course-add-semester
branch
from
ea4c8b6 to
549e78e
Compare
skkuding-bot
Bot
added
preview
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
seoeun9
left a comment
seoeun9
left a comment
There was a problem hiding this comment.
고생하셨어요!! 전체적인 디자인은 좋은 거 같습니다. 다만 용어 정리가 좀 필요할 거 같아요!
-
Course Number가 Course code랑 같은 섹션인 거 같은데, 강의명이 나오게 되어있더라구요! SWE1999 같은 course code가 복제되게 해야할 듯합니다. 헷갈릴 수 있으니 타이틀도 Course Code로 통일하고, 입력 7자리 제한도 가능하면 넣어주시면 좋을 거 같아요
-
마찬가지로 Class Number는 Course section(분반)에 해당하는 거 같은데 2자리 제한 + 타이틀 통일하면 어떨까요..ㅎㅎ
seoeun9
left a comment
seoeun9
left a comment
There was a problem hiding this comment.
여기 입력 필드도 살짝 가려지는..? 문제가 있습니다!
|
@seoeun9 디자인은 수정하겠습니다! |
seoeun9
left a comment
seoeun9
left a comment
There was a problem hiding this comment.
@sanghyeonjung 넵 표시되는 것만 바꿔주세요! ㅎㅎ
|
✅ Syncing Preview App Succeeded Application: |
Choi-Jung-Hyeon
left a comment
Choi-Jung-Hyeon
left a comment
There was a problem hiding this comment.
머지 컨플릭나는거 있으면 해결부탁요!
|
✅ Syncing Preview App Succeeded Application: |
seoeun9
left a comment
seoeun9
left a comment
There was a problem hiding this comment.
오랫동안 수정하느라 고생 많으셨습니다.. 큰 문제 없는 거 같고 굿굿이에요!!
semester 쪽 로직은 안 그래도 따로 utils로 빼면 어떨까 했는데, 이건 태스크를 새로 하나 파보아요 👏
감사합니다! 중현님도 확인하시면 머지하겠습니다. |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
❗ Syncing Preview App Failed Application: |
|
❗ Syncing Preview App Failed Application: |
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
github-merge-queue
Bot
removed this pull request from the merge queue due to a conflict with the base branch
|
✅ Syncing Preview App Succeeded Application: |
3 participants
Description
admin 에서 duplicate course 를 눌렀을 때, 입력 칸이 박스 밖으로 튀어 나오는 문제를 스크롤 바를 추가해 수정했습니다. 기존에는 input 데이터 형식을 고려하지 않고 무조건 duplicate 하도록 설계되어 있었는데, 이제 입력 형식이 맞지 않으면 빨간색 글씨가 표기 되게 변경했습니다.
Additional context
디자인이 없었던 태스크라 혹시 문제가 있으면 말해주시면 좋겠어요! 또, 이런 기능적인 내용을 추가하는 태스크가 처음이라 혹시 제대로 돌아가더라도 비효율적인 부분이나 개선하면 좋겠는 부분은 얼마든지 말씀해주시면 감사하겠습니다!
+코드 리뷰 반영(5/4)
useMemo를 사용해 courseNum, semester, classNum 값이 변경될 때마다 유효성을 검사합니다.
유효하지 않은 경우 courseNumError, semesterError, classNumError에 에러 메시지를 저장하고, 유효한 경우 빈 문자열로 저장합니다. 에러가 있을 경우 해당 입력란 하단에 빨간색으로 표시되며, 모든 값이 유효할 때만 Duplicate 버튼이 활성화됩니다.
primaryButton에 disabled 속성이 없어 ButtonProps 인터페이스에 선택 속성으로 추가했습니다.
중현님 요청에 따라 모달 크기를 md에서 lg로 변경했습니다. 이전과 비교해보시고 피드백 주시면 감사하겠습니다!
semester 입력 방식을 텍스트 입력에서 드롭다운으로 변경했습니다. CourseFormFields에 작성된 우진님의 코드와 동일한 로직을 사용했습니다.
++코드 리뷰 반영 (5/6)
course code 와 class number 를 create course 에서 사용하는 용어와 동일하게 수정하였습니다. course code 에는 7글자 제한을 추가했고, class number 에는 2글자 제한을 추가했습니다.
Before submitting the PR, please make sure you do the following
fixes #123).