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

[Fix/#462] 자잘한 디자인 QA + 변경사항 반영 #473

Merged
merged 13 commits into from
Dec 30, 2024

Conversation

sinji2102
Copy link
Contributor

@sinji2102 sinji2102 commented Dec 26, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

예매자 관리 페이지

  • 모바일에서 필터 svg 찌그러지는 오류 해결

  • 예매자 관리 페이지 내 empty view일 때 cta 없도록 변경

  • 필터 체크박스 컬러 500 -> 400

  • 적용하기 비활성화 시 cta 글자 색 white -> gray 500 (disable 상태 변경)

  • 가장 마지막 카드가 cta 버튼에 가려져 안 보이는 오류 해결

  • 예매자 관리 상태 시 뒤로가기 버튼 클릭하면 페이지 뒤로 가는 게 아닌 DEFAULT 상태로 변경

  • 환불 계좌 복사 시 은행까지 복사

  • 예매자 없을시에는 필터링 버튼 삭제 + 검색바 끝까지 채우기

  • 검색바 플레이스홀더 포커스 시 멘트 변경

  • 스크롤 위로 쭉 당겼을 때 cta 버튼 양옆으로 여백 나오는 오류 해결

공연 관리 페이지

  • 버튼 텍스트 중앙 정렬 필요

📢 To Reviewers

  • 스크롤 위로 쭉 당겼을 때 cta 버튼 양옆으로 여백 나오는 오류 해결
    -> 카카오웹 이슈로 확인했습니다...!!

  • 버튼 텍스트 중앙 정렬 필요
    image
    -> 확인해봤는데 중앙 정렬이 아니라 폰트 이슈인 것 같아요... 지금 메인도 다 글지 위로 치우쳐져 있고 처음엔 정렬 맞는데 3초 뒤에 안 맞음 ㅠㅠ

  • 모바일에서 필터 svg 찌그러지는 오류 해결
    -> 일단 해결해보긴 했는데 데스크탑에선 확인이 불가능해서 머지하고 다시 확인하겠습니다!

@sinji2102 sinji2102 linked an issue Dec 26, 2024 that may be closed by this pull request
10 tasks
@github-actions github-actions bot added the 🐛 BUG 버그를 고친 경우 label Dec 26, 2024
Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

확인했쑵니다!!

@@ -29,6 +29,7 @@ import { IconCheck } from "@assets/svgs";
import Toast from "@components/commons/toast/Toast";
import { useToast } from "@hooks";
import NonExistent from "./components/nonExistent/NonExistent.";
import { filter } from "lodash";
Copy link
Member

Choose a reason for hiding this comment

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

p4: 얘는 언제 쓰는거죵 ?

Copy link
Contributor

@imddoy imddoy left a comment

Choose a reason for hiding this comment

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

굿굿굿

@sinji2102 sinji2102 merged commit 50d0a19 into develop Dec 30, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 BUG 버그를 고친 경우
Projects
Status: 🎉Done
Development

Successfully merging this pull request may close these issues.

[ Bug ] 자잘한 디자인 QA + 변경사항 반영
3 participants