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
- 추후 변동 가능성 있음.
PHJoon
approved these changes
Aug 11, 2023
components/modal/ModalButton.tsx
Outdated
…into GGFE-153-상점-확성기-모달
hyobb109
reviewed
Aug 11, 2023
There was a problem hiding this comment.
주의사항 컴포넌트도 따로 분리해두니까 편할 것 같네요!👍
hyobb109
reviewed
Aug 11, 2023
| const MAX_LENGTH = 30; | ||
|
|
||
| // TODO : 주의사항 구체화 필요 | ||
| const caution = [ |
There was a problem hiding this comment.
23:55 - 00:05 사이엔 등록 불가하다는 문구도 나중에 추가하면 좋을 것 같아요
hyobb109
reviewed
Aug 11, 2023
| rows={1} | ||
| name='newMegaphoneContent' | ||
| onChange={(e) => setContent(e.target.value)} | ||
| maxLength={MAX_LENGTH} |
There was a problem hiding this comment.
한글은 바이트 수가 달라서 그런지 31자까지도 받네요ㅠ 찾아보니까 정규식으로 변환해서 length를 딱 맞게 구할 수 있다는데 중요한 부분인지는 잘 모르겠어요 ㅎㅎ 근데 27자 넘어가면 모달이 점점 넓어져서 max-width 스타일 설정해도 좋을 것 같아요!
There was a problem hiding this comment.
모달이 점점 넓어지는 문제는 입력창 부분에 max-width를 지정하면 될 듯 한데 해보니까 이게 딱 27자 기준은 아니고 화면 너비에 따라서 모달이 넓어지는 포인트가 다르네요... 이부분은 일단 이슈로 남겨두고 계속 방법 고민해보도록 하겠습니다 ^_ㅜ
hyobb109
approved these changes
Aug 11, 2023
|
등록 불가능한 시간에 대한 주의사항 문구 추가했고 |
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.
📌 개요
💻 작업사항
확성기
확성기 미리보기 기능을 위해 확성기 컴포넌트를
Megaphone컴포넌트와MegaphoneContainer컴포넌트로 분리했습니다.혹시 몰라서 원래 코드는 그대로 주석처리 해 두었습니다. 테스트 후에 문제 없으면 머지 전에 주석은 삭제할게요.
모달 버튼 컴포넌트 분리
일반적인(?) 모달 하단 버튼으로 사용할 수 있는
ModalButtonContainer,ModalButton컴포넌트를 만들었습니다.positive스타일과negative스타일이 있고 버튼은 1개나 2개 넣을 수 있어요.테스트 user 정보 변경
recoil에 저장되어 있는 유저 정보가 mock api의 응답이다 보니까 의도치않은 결과가 발생할수도 있을 것 같아서 intra id를 존재하지 않는 유저 kim_takgu로 변경했습니다. 실제 존재하는 유저로 테스트가 필요한 경우에는 이름 변경해서 테스트하면 됩니다.
확성기 등록 모달
보관함 데이터에서 각 아이템 코드에 해당하는 모달을 띄우는 방식입니다. (아이템 코드는 아직 확정되지 않아서 임의로 넣었습니다.)
원래 디자인으로는 미리보기 버튼을 눌러서 실제 보여지는 결과를 확인하는 것으로 되어 있었는데 입력과 동시에 바로 보여주는 것이 디자인적으로나 기능적으로나 좀 더 깔끔한 것 같아서 바로 미리보기 결과를 확인할 수 있게 했습니다.
등록 결과를 바로 확인할수가 없어서 일단 테스트용으로 새로 등록한 확성기 내용을 배열에 추가해서 응답으로 보내고 콘솔에 출력했습니다
모든 아이템 코드를 확성기로 해 두었기 때문에 테스트는 모든 사용하기 버튼으로 할 수 있어요.
✅ 변경로직