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

Refctor/#148 qa design #149

Merged
merged 8 commits into from
Jan 1, 2025
Merged

Refctor/#148 qa design #149

merged 8 commits into from
Jan 1, 2025

Conversation

seohee0925
Copy link
Contributor

@seohee0925 seohee0925 commented Dec 29, 2024

📌 PR 요약

기획 & 디자인 1차 QA 수정

🌱 작업한 내용

  • 탭바 상단 라인 두께 및 컬러값 안맞은거 수정
  • 로그인화면의 비스킷 로고 크기와 위치 안맞은거 수정
  • Title 워딩 수정
  • ‘후기 영상’ 탭의 ‘아직 후기가 없어요~후기를 공유해보세요’ 텍스트 박스 위치 안맞은거 수정
  • 왼쪽 상단에 x 아이콘 구현 (뒤로 가기와 같도록 함)
  • 첫번째 텍스트박스에 입력중일 때 컬러값 안맞은거 수정 (yellow에서 transparent로 함)
  • 위치칩 디자인 안맞은거 간격으로 수정
  • 신고하기- 기타’ 항목에서 ‘완료’ 버튼 비활성화 (아무 텍스트도 작성하지 않았을 때)
  • n개의 기록 텍스트 박스와 하단 영상 카드 간격 안맞은거 수정
  • ‘영상 업로드하기’ 버튼 텍스트 수정
  • ‘아직 팔로워가 없어요’ 모달창 그래픽 에셋 변경

🌱 PR 포인트

  • dp → sp 로 수정 불가(수정하면 theme 안에서 에러 발생) : 앱 내 텍스트 크기 고정 필요
  • 장소 카드 간 간격 안맞은거 16dp 다 동일하게 설정했는데 영상 칸이랑 전시회 명이 있는 칸 간의 차이가 존재
  • 왼쪽 상단에 x 아이콘 구현한거 확인 필요
  • 신고하기- 바텀시트 핸들바 색상 안맞은거 피그마 색상이랑 theme에 있는 색상 color 다 비교해봤는데 동일.. → 확인 필요
  • 프로필탭-내가 팔로잉한 사람의 목록이 ‘팔로잉’이 아닌 ‘팔로워’에 기록 되는데, api 호출 및 코드는 다 올바름.. → 확인 필요
  • 프로필 수정 완료 후 프로필 창으로 연결(현재 설정 창으로 연결됨)하는거 계속 시도 중 → 현재 문제점) profile Id가 전달되어야 해서 sideEffect.id로 보내려고 했으나 id가 전달이 안됨

📸 스크린샷

📮 관련 이슈

제가 작성한 코드가 대부분 아니다 보니까 조심스럽게 건드렸어요 😿

@seohee0925 seohee0925 added the 서희 🐶🪽 안녕하세요 떠희입니다 Skrrrrr~ label Dec 29, 2024
@seohee0925 seohee0925 self-assigned this Dec 29, 2024
Copy link
Contributor

@Sangwook123 Sangwook123 left a comment

Choose a reason for hiding this comment

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

Merge gogo

@seohee0925 seohee0925 merged commit f8845f3 into develop Jan 1, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
서희 🐶🪽 안녕하세요 떠희입니다 Skrrrrr~
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] 디자인 QA 수정
2 participants