kuring-95 알림 구독 화면 2.0 디자인 적용 #84
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
UI 코드는 나중에 알림 수정 화면 migration이 끝나면 삭제할 예정이다.
함수 파라미터 상 도메인 Department를 가지고 있어야 하는 경우가 있어서, 일부 변수에는 ui model 대신 Department 클래스를 사용하였다.
구독 편집 화면에서 보여줄 데이터를 담고 있는 UiState 클래스를 추가하였다.
텍스트, 일반적인 composable 둘 다 action으로 사용할 수 있도록 오버로딩하였다.
모든 모듈이 24로 설정되어 있는데, 혼자서만 23으로 설정되어 있던 오류를 수정했다.
mwy3055
commented
Jan 5, 2024
| @@ -36,12 +38,11 @@ class EditSubscriptionViewModel @Inject constructor( | |||
|
|
|||
| private val disposable = CompositeDisposable() | |||
|
|
|||
| private val initialCategories = MutableStateFlow(mutableSetOf<SubscriptionUiModel>()) | |||
| private val initialCategories = MutableStateFlow(emptyList<NormalSubscriptionUiModel>()) | |||
| private val initialDepartments = MutableStateFlow(mutableSetOf<Department>()) | |||
There was a problem hiding this comment.
어차피 순서대로 보여줄 거라 굳이 Set을 사용할 필요가 없다고 생각했습니다. (학사 -> 취창업 -> ...)
mwy3055
commented
Jan 5, 2024
...scription/src/main/java/com/ku_stacks/ku_ring/edit_subscription/EditSubscriptionViewModel.kt
Outdated
Show resolved
Hide resolved
mwy3055
commented
Jan 5, 2024
| val CaptionGray1: Color | ||
| @Composable get() = Color(0xFF868A92) | ||
| val CaptionGray2: Color | ||
| @Composable get()=Color(0xFFAEB1B6) |
There was a problem hiding this comment.
Figma에 이름이 등록되어 있는 색깔이라 변수로 추가했습니다.
mwy3055
commented
Jan 5, 2024
| ) | ||
| } | ||
| } | ||
| } | ||
|
|
||
| private fun onSubscriptionComplete() { |
There was a problem hiding this comment.
기존에 구현되어 있던 전환 로직을 함수로 분리해서 그대로 사용했습니다.
yeon-kyu
reviewed
Jan 8, 2024
...scription/src/main/java/com/ku_stacks/ku_ring/edit_subscription/EditSubscriptionViewModel.kt
Outdated
Show resolved
Hide resolved
...bscription/src/main/java/com/ku_stacks/ku_ring/edit_subscription/EditSubscriptionActivity.kt
Outdated
Show resolved
Hide resolved
...bscription/src/main/java/com/ku_stacks/ku_ring/edit_subscription/EditSubscriptionActivity.kt
Outdated
Show resolved
Hide resolved
MainActivity에서 간결한 Composable을 호출하고, 간결한 Composable 내부에서 장황한 Composable을 호출하도록 수정했다.
yeon-kyu
reviewed
Jan 17, 2024
...esignsystem/src/main/java/com/ku_stacks/ku_ring/designsystem/components/CenterTitleTopBar.kt
Show resolved
Hide resolved
...esignsystem/src/main/java/com/ku_stacks/ku_ring/designsystem/components/CenterTitleTopBar.kt
Show resolved
Hide resolved
...esignsystem/src/main/java/com/ku_stacks/ku_ring/designsystem/components/CenterTitleTopBar.kt
Show resolved
Hide resolved
...signsystem/src/main/java/com/ku_stacks/ku_ring/designsystem/components/PreviewAnnotations.kt
Show resolved
Hide resolved
common/ui_util/src/main/java/com/ku_stacks/ku_ring/ui_util/KuringNavigator.kt
Show resolved
Hide resolved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://kuring.atlassian.net/browse/KURING-95?atlOrigin=eyJpIjoiYTgwNGUyYzk3NmZiNDVjYmFkYmI5NDcwNmM0YjRhZGUiLCJwIjoiaiJ9
알림 구독 화면에 2.0 디자인을 적용했습니다.
_2024_01_05_21_44_08_79.mp4
구현 상세
theme/Color.kt파일에 이름으로 추가했고, 그렇지 않은 경우에는Color(0xFFFFFFFF)식으로 하드코딩했습니다. 향후 테마가 완성되면 수정할 예정입니다.:common:designsystem모듈에 구현했습니다. (Top app bar 등)그 밖에 궁금하신 점이 있다면 코멘트로 남겨 주세요.
변경사항
3670f13: 일반 카테고리 화면에서 사용할 아이콘 png 파일을 추가했습니다.
ead00bf: 알림 구독 레이아웃 파일에서
ComposeView외의 모든 뷰를 제거했습니다.afe2a77: 일반 카테고리 구독 화면에서, 카테고리 하나를 나타내는 ui model 클래스를 추가했습니다.
4592ef3:
ViewModel에서 ui model 클래스를 활용하여 일반 카테고리 데이터를 노출하도록 수정했습니다.966d4bf: (:common:designsystem) Compose Preview 어노테이션을 추가했습니다.
22ec52a: 일반 카테고리 하나를 구독하는 composable을 작성했습니다.
0641625: 학과 카테고리 구독 화면에서, 학과 하나를 나타내는 ui model 클래스를 추가했습니다.
7710694: 학과 카테고리 하나를 구독하는 composable을 작성했습니다.
c4b3cd9:
ViewModel에 학과 카테고리를 구독하는 비즈니스 로직 함수를 작성했습니다.5aa1cb0: 구독 화면을 구현하던 레거시 composable 코드를 제거했습니다.
aabe3c6: 일반/학과 카테고리 탭을 나타내는 데이터 클래스를 작성했습니다.
8ae4ea6: 알림 구독 화면에서 사용할 UiState 클래스를 작성했습니다. 이 클래스는
ViewModel의StateFlow에 의해 UI에 노출됩니다.29d430c: 구독 화면 composable이 UiState 클래스를 매개변수로 받도록 수정했습니다.
82e5bc7: (:common:designsystem) 2.0 디자인에서 자주 사용되는 Top app bar를 추가했습니다.
1c23aed: (gradle)
:common:designsystem모듈의minSdk를 23에서 24(다른 모든 모듈의 minSdk)로 수정했습니다. 모듈을 만들 때 빠트린 듯합니다.b3337bf: (:common:designsystem) 2.0 디자인에서 흔히 사용될 CTA 버튼을 구현했습니다.
28d0763: 알림 구독 화면 composable 전체를 구현했습니다.
825f843: 알림 구독 화면 composable을
Activity와 연결했습니다.ed328ce: 공지 구독 화면을 띄우는 함수를
KuringNavigation에 추가했습니다. 앱을 처음 실행했을 때, 알림 구독 화면에서 공지 구독 창으로 갈 수 있게 하기 위해서입니다.8dfc30a: 알림 구독 화면에서 공지 구독 화면으로 가는 코드를 작성했습니다.
83c1a3a: 공지 구독 화면에서 구독한 학과가 알림 구독 화면에서 보이지 않던 문제를 해결했습니다.
6f2b28d: 알림 구독 화면에서 선택한 학과가 알림 구독되지 않던 문제를 해결했습니다.
ebabe5f: 레거시 코드 중 더 이상 사용하지 않는 코드를 제거했습니다.