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

[Feat] 관리자 전체 유저 알림 모달 개선 #650

Closed
7 tasks done
Wilbur0306 opened this issue Feb 13, 2023 · 0 comments
Closed
7 tasks done

[Feat] 관리자 전체 유저 알림 모달 개선 #650

Wilbur0306 opened this issue Feb 13, 2023 · 0 comments
Assignees
Labels
admin 관리자 페이지에서 발생한 issue enhancement New feature or request

Comments

@Wilbur0306
Copy link
Contributor

Wilbur0306 commented Feb 13, 2023

Desc

관리자 전체 유저 알림 모달에서 부족한 부분 해결

Detail

CSS, ModalProvider 등 문제점 개선

Todo

  • [기능] 관리자 페이지 "알림 관리" 부분에 있는 "All" 버튼과 모달 연결해주기
  • [CSS] 적용, 취소 버튼 모두 크기 크게 수정
  • [CSS] [적용] 버튼을 누른 후, [적용] 버튼이 사라지는 것이 아니라, "토스트 메세지"로 전송 여부를 확인할 수 있게끔 수정
  • [CSS] input 입력창 안에서 입력이 가운데가 아닌 좌측상단에서부터 시작되도록 수정
  • [CSS] 관리자 페이지의 다른 디자인들과 통일
  • [CSS] input 입력창 안에서 입력시 자동 줄바꿈 + 입력창을 넘어가지 않도록 수정
  • [ModalProvider] 전체 유저에게 보내는 부분인데 intra ID 불필요한데 작성되어 있는 것 수정
@Wilbur0306 Wilbur0306 added enhancement New feature or request admin 관리자 페이지에서 발생한 issue labels Feb 13, 2023
@Wilbur0306 Wilbur0306 self-assigned this Feb 13, 2023
@Wilbur0306 Wilbur0306 moved this from 🆕 New to 🏗 In progress in 🍌 Version 2 - Frontend Feb 13, 2023
@Wilbur0306 Wilbur0306 linked a pull request Feb 20, 2023 that will close this issue
2 tasks
@Wilbur0306 Wilbur0306 moved this from 🏗 In progress to 👀 In review in 🍌 Version 2 - Frontend Feb 20, 2023
Wilbur0306 added a commit that referenced this issue Feb 21, 2023
mike2ox added a commit that referenced this issue Feb 25, 2023
- 불필요한 useState, 함수들 제거
- useRef를 사용해 불필요한 render현상 막음
- 다음 미션 주석 작성
mike2ox added a commit that referenced this issue Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin 관리자 페이지에서 발생한 issue enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant