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 ] junior promise page QA 반영 #340

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

yarimu
Copy link
Contributor

@yarimu yarimu commented Nov 4, 2024

#️⃣ Related Issue

Closes #338

✅ Done Task

  • 바텀시트 클릭됐을때만 '적용할래요'버튼 커서 포인터 적용
  • 클릭 가능한 모든 것 커서 포인터 적용
  • 스크롤바 보이는 거 삭제
  • 바뀐 헤더 적용

언제나 정신 똑바로 차리고 살아야 합니다..
헤더를 수정해도 반응조차 없길래.. 억장이 무너져 내렸는데 알고보니 지은이 뷰랑 연결된 헤더만 열심히 수정하고
확인은 제 뷰에서 했으니.. 얼마나 바보같은 짓을 하고 있었는지..
📢 지은이에게
하는김에 지은이 뷰 헤더도 수정했는데 피그마 상에 지은이 뷰는 헤더가 없더라구요?? 조건부 렌더링에는 지은이가 헤더를 해두었길래
일단 "transparent"헤더로 해두었는데.. 지은이 한 번 확인 부탁드림다~ (그래서 지은이 소환했어요)

💎 PR Point

커밋 순서대로, 커서 포인터 적용했구(적용할래요 버튼 색 들어오는 경우에만 커서포인터 수정했습니다),
바텀시트 직무부분 스크롤 안보이게 수정완료했습니다!
바뀐 헤더대로 수정 해두었습니다.

📸 Screenshot

  • 커서포인터
default.mp4
  • 스크롤 수정
default.mov
  • 바뀐 헤더
default.mov

@yarimu yarimu added the 🛠 Fix 기존의 버그 수정 label Nov 4, 2024
@yarimu yarimu self-assigned this Nov 4, 2024
@github-actions github-actions bot added the size/m label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Fix ] JuniorPromisePage 커서 포인터 적용 및 스크롤바 없애기
1 participant