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

[Refactoring] 기타 자잘하게 수정 필요한 사항들 #270

Open
8 of 11 tasks
xohxe opened this issue Feb 23, 2024 · 2 comments
Open
8 of 11 tasks

[Refactoring] 기타 자잘하게 수정 필요한 사항들 #270

xohxe opened this issue Feb 23, 2024 · 2 comments
Assignees
Labels
리팩토링 리팩토링할때

Comments

@xohxe
Copy link
Collaborator

xohxe commented Feb 23, 2024

리팩토링

리팩토링 할 코드에 대한 설명

요구사항

  • UX개선
  • 기타 오류들을 확인하고 고쳐내자.

Todo

  • 메인 지도뷰에서 리스트뷰 클릭 시, 리스트만 보여주기 -> 이에따른 커뮤니티뷰 탭 분리
  • 빌딩명 수정하기
  • 핀 체크되면 기울어지게 애니메이션 주면 좋을듯
  • 영덕대게 폰트 Import 안된거 확인 및 새로운 폰트추가하기(용량작은거로)
  • 메모지 추가
  • 디테일뷰에서 좋아요 수 표기
  • 디테일뷰에서 저장 버튼 제거
  • 카드뷰에서 하트부분에 프로필이미지들어가게 변경?
  • 프로필이미지 없을 때 default 이미지 만들어주기
  • 프로필 이미지 변경시, 기본이미지로 바꿔주는 기능도 있어야함.
  • Postview에서 업로드 한 이미지가 찌그러지는 현상

프로필이미지 없을 때 default 이미지 만들어주기

스크린샷 2024-02-23 오후 4 26 36

url이 빈 필드일 경우까지 예외처리 해주어야하므로, 다음과 같이 수정해주었습니다.

// MemoCard.swift
 if let url = profile.profilePicture, !url.isEmpty {
      if url.count > 0 {
          Button /* NavigationLink */ {
              completion(.navigate(profile: profile))
          } label: {
              KFImage(URL(string: url))
                  .resizable()
                  .scaledToFill()
                  .clipped()
                  .clipShape(.circle)
                  .frame(width: 37,height: 37)
                  .contentShape(Circle())
                  .cornerRadius(19)
          }
      }
      
  } else {
      Image("profileImg")
          .frame(width: 37,height: 37)
          .clipShape(.circle)
          
  }
@xohxe xohxe added the 리팩토링 리팩토링할때 label Feb 23, 2024
@xohxe xohxe self-assigned this Feb 23, 2024
@Ahario
Copy link
Collaborator

Ahario commented Feb 24, 2024

개인적으로 설정 -> 폰트에서 폰트 변경리스트에서 박스를 터치해서 변경이 아닌 그 폰트 항목을 클릭시 변경되는걸로 고치면 편할것같습니다.

@xohxe
Copy link
Collaborator Author

xohxe commented Feb 24, 2024

개인적으로 설정 -> 폰트에서 폰트 변경리스트에서 박스를 터치해서 변경이 아닌 그 폰트 항목을 클릭시 변경되는걸로 고치면 편할것같습니다.

오 좋습니다. 해당 부분도 반영해볼게요 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
리팩토링 리팩토링할때
Projects
None yet
Development

No branches or pull requests

2 participants