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 수정 #125

Merged
merged 5 commits into from
Mar 22, 2024

Conversation

eung7
Copy link
Contributor

@eung7 eung7 commented Mar 22, 2024

Screenshots 📸

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-03-22.at.17.55.39.mp4



고민, 과정, 근거 💬

  • 제품 상세 화면에 스크롤 뷰를 임베드했습니다.
  • 제품 상세 화면 원가격에 취소선을 추가했습니다.
  • 서버에서 불러온 상품 데이터들의 이미지 URL이 종종 null로 응답되는 경우가 있어서 이를 다음과 같이 수정했습니다.
    • 이미지URL이 null일 경우 ProgressView가 무한로딩이 되는 버그를 편행 로고 이미지로 바꾸었습니다. (홈, 검색)
    • 제품 상세 화면에서 이미지 URL이 null일 경우 아무것도 뜨지 않는 버그를 수정했습니다.
  • 검색 화면에서 백 스와이프로 이전 화면으로 넘어가는 기능을 추가했습니다.
    • 이 부분은 기존 커스텀 네비게이션 바를 사용했지만, UIKit 코드를 접합시켜 백 스와이프 기능을 추가할 수 있었습니다. 하지만 많은 부분에서 버그가 발생하더군요.. ㅠㅠ
    • 그래서 다른 방안으로 네비게이션 바를 임베드하고 깨지지 않을 정도의 높이를 가지고 텍스트 필드를 추가했습니다.



References 📋




@eung7 eung7 added 🏪 Products Products View 🔨 Implementation New feature or request 🔍 Search Search View labels Mar 22, 2024
@eung7 eung7 added this to the v2.0.0 milestone Mar 22, 2024
@eung7 eung7 requested a review from a team March 22, 2024 09:05
@eung7 eung7 self-assigned this Mar 22, 2024
@eung7 eung7 linked an issue Mar 22, 2024 that may be closed by this pull request
Copy link
Member

@WhiteHyun WhiteHyun left a comment

Choose a reason for hiding this comment

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

많은 고민을 하셨네요. 정말 고생 많으셨습니다.

@WhiteHyun WhiteHyun merged commit 3c561ff into main Mar 22, 2024
8 checks passed
@eung7 eung7 deleted the feature/productinfo/122 branch March 22, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Implementation New feature or request 🏪 Products Products View 🔍 Search Search View
Projects
Development

Successfully merging this pull request may close these issues.

제품 상세 정보 미비된 UI 수정
2 participants