Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

가이드라인, 시니어 추가 페이지 UI 수정 #122

Merged
merged 2 commits into from
Oct 28, 2024

Conversation

Diwoni
Copy link
Collaborator

@Diwoni Diwoni commented Oct 27, 2024

#️⃣ 연관된 이슈

ex) #113

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

UI 수정

  • 괜찮은 디자인이 생각나지 않아 일단 위치 조정과 등록되어있는 가이드라인,시니어 수를 보여주도록 추가했습니다.

수정 모드에서 취소 버튼 추가

  • 수정 시 취소버튼이 없었어서 취소 버튼 추가했습니다. 수정 모드에서 이제 수정 취소 버튼으로 수정 모드를 취소할 수 있습니다.

삭제 버튼 기능

  • 삭제 버튼 누를 시 사용자가 실수로 눌렀을 때를 대비해 window.confirm 메서드로 재차 확인시킬 수 있도록 했습니다.

스크린샷 (선택)

수정된 UI

image

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

⏰ 현재 버그

✏ Git Close

close #113

@Diwoni Diwoni added the 🎨 Design 디자인 관련 작업을 진행하는 경우 label Oct 27, 2024
@Diwoni Diwoni requested review from Dobbymin and JYN523 October 27, 2024 18:40
@Diwoni Diwoni self-assigned this Oct 27, 2024
@Diwoni Diwoni linked an issue Oct 27, 2024 that may be closed by this pull request
Copy link
Contributor

@Dobbymin Dobbymin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

저는 멘토님의 이 말씀을 'api 관련 로직 이외에 다른 로직들도 커스텀 훅으로 작성할수 있을것 같다.' 라고 이해했는데, 제가 잘못이해했을수도 있어서 이 부분은 잘 생각해보고 판단하면 될 것 같네요!
일단 큰 이슈 없는것 같아서 일단 approve 눌러놓겠습니다.

Comment on lines 66 to +70
const Container = styled(Box)`
position: relative;
height: 100vh;
display: flex;
flex-direction: column;
align-items: center;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

멘토님이 이렇게 chakra UI 를 styled로 정의하지 말라고 하시더라구요 ㅋㅋ.. 수정해야할듯합니다

@Diwoni Diwoni merged commit 4779e98 into Weekly Oct 28, 2024
1 check passed
@Diwoni Diwoni deleted the Style/issue-#113 branch October 28, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Design 디자인 관련 작업을 진행하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

가이드라인 페이지 시니어 목록 페이지 UI 수정
2 participants