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

[2.0.0] 다크모드 전부 적용 및 학과 공지 UI 버그 및 다크모드 색상 적용을 위한 UI 수정 #158

Merged
merged 11 commits into from
Mar 24, 2024

Conversation

lgvv
Copy link
Member

@lgvv lgvv commented Mar 21, 2024

[2.0.0] 다크모드 전부 적용 및 학과 공지 UI 버그 및 다크모드 색상 적용을 위한 UI 수정

  • 앱 내 다크모드를 전부 적용했습니다.
  • 학과 공지사항 UI가 깨지던 이슈를 수정했습니다.
    • 수정 UI는 카카오톡으로 전달드렸습니다!
  • UI 간격 조정을 위해 ScrollView + LazyVStack을 사용했었는데, List로 변경 가능할 것으로 보입니다.
    • SwiftUI의 List의 성능이 좋아졌다고 피드백 주셔서 이 부분은 후속으로 챙기겠습니다.
  • 온보딩 등 다크모드를 제작하면서, UI에 문제가 발생하던 부분을 전부 해결했습니다.

@lgvv lgvv changed the title [2.0.0] 다크모드 전부 적용 및 학과 공지 UI 이슈 [2.0.0] 다크모드 전부 적용 및 학과 공지 UI 버그 및 다크모드 색상 적용을 위한 UI 수정 Mar 21, 2024
@lgvv lgvv added 학과 학과 관련 기능 (학과 검색, 내학과) v2 공지리스트 공지디테일, 공지리스트 기능 디자인시스템 디자인 시스템 labels Mar 21, 2024
@lgvv lgvv requested review from x-0o0 and sunshiningsoo March 21, 2024 16:02
@lgvv lgvv linked an issue Mar 21, 2024 that may be closed by this pull request
@lgvv lgvv self-assigned this Mar 21, 2024
Copy link
Contributor

@x-0o0 x-0o0 left a comment

Choose a reason for hiding this comment

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

👍LGTM

@sunshiningsoo
Copy link
Member

대박!!!

Copy link
Member Author

@lgvv lgvv left a comment

Choose a reason for hiding this comment

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

공통UI로 빼낼 수 있는 것들 빼내기

  • 셀프 리뷰

@@ -229,6 +242,7 @@ public struct SearchView: View {

public init(store: StoreOf<SearchFeature>) {
self.store = store
// UITableView
Copy link
Member Author

Choose a reason for hiding this comment

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

불필요한 코드 제거하기👀

@x-0o0
Copy link
Contributor

x-0o0 commented Mar 24, 2024

/쿠링 패키지 테스트

Copy link

🔨 iOS 17 iPhone 15 Pro 에서 쿠링 패키지를 테스트 합니다.

Copy link

📬 iOS17 에서 패키지 테스트에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@lgvv lgvv enabled auto-merge March 24, 2024 04:34
@lgvv lgvv disabled auto-merge March 24, 2024 04:34
@lgvv lgvv merged commit 2892203 into develop/2.0.0 Mar 24, 2024
@lgvv lgvv deleted the feature/lgvv/ui-darkmode branch March 24, 2024 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2 공지리스트 공지디테일, 공지리스트 기능 디자인시스템 디자인 시스템 학과 학과 관련 기능 (학과 검색, 내학과)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[학과 공지] 학과 공지 선택 UI가 깨지는 버그
3 participants