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

홈 탭바 디자인 변경 #241

Merged
merged 5 commits into from
Sep 30, 2024
Merged

홈 탭바 디자인 변경 #241

merged 5 commits into from
Sep 30, 2024

Conversation

fbghgus123
Copy link
Collaborator

개요

이슈 링크 혹은 PR 내용 요약

홈 탭바 디자인 변경 반영

작업 내용

실제 작업 내용

  • 홈 탭바 블러 삭제
  • 홈 탭바 아이콘 삭제

시연 화면 (option)

실행 스크린샷 혹은 영상 첨부

홈 탭바
image

To Reviers

리뷰어들에게 전할 말

v2 반영 안된 것들 반영 햇습니당

Close

close #

@@ -171,25 +170,17 @@ class HomeViewModel @Inject constructor(
intent {
reduce {
state.copy(
tapElements = folderList.mapIndexed { index, folder ->
tapElements = folderList.mapIndexed { _, folder ->
Copy link
Collaborator

Choose a reason for hiding this comment

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

안쓰면뭐 고냥 map이여도 될 듯? 바꾸긴 귀찮으니,, 그렇다고 ..

Copy link
Collaborator Author

@fbghgus123 fbghgus123 Sep 30, 2024

Choose a reason for hiding this comment

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

바꾸겟음 ㅋㅋ(나중에)

Copy link
Member

@ddyeon ddyeon left a comment

Choose a reason for hiding this comment

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

고생했슈

@fbghgus123 fbghgus123 merged commit 611cb50 into develop Sep 30, 2024
3 checks passed
@fbghgus123 fbghgus123 deleted the chore/home_tap branch September 30, 2024 12:15
@fbghgus123 fbghgus123 restored the chore/home_tap branch October 3, 2024 12:44
@Ahn-seokjoo Ahn-seokjoo deleted the chore/home_tap branch October 20, 2024 15:06
@Ahn-seokjoo Ahn-seokjoo restored the chore/home_tap branch October 20, 2024 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants