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

[1.2.0/AN-UI, AN-FEAT] 특성 디테일 스크롤 시 앱 바 애니메이션 적용 #466

Merged
merged 22 commits into from
Dec 6, 2024

Conversation

sh1mj1
Copy link
Contributor

@sh1mj1 sh1mj1 commented Nov 27, 2024

작업 영상

Screen.Recording.Nov.27.2024.mov

작업한 내용

  • 특성 상세에서 스크롤 시 애니메이션
    (아래로 스크롤 시에 상단 앱바가 사라졌다가 위로 스크롤 하면 상단 앱바가 다시 나타난다.)
  • 가로 화면도 대응

PR 포인트

🚀Next Feature

@github-actions github-actions bot added AN_FEAT ✨ 안드 새로운 기능 AN_UI 🎨 안드 UI 작업 labels Nov 27, 2024
Copy link
Contributor

@murjune murjune left a comment

Choose a reason for hiding this comment

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

LGTM 제목이 특성 목록 스크롤~이라 특성 디테일 스크롤 앱 바 애니메이션 적요으로 바꿀게요~

Comment on lines +19 to +42
<androidx.coordinatorlayout.widget.CoordinatorLayout
android:layout_width="match_parent"
android:layout_height="match_parent"
tools:context=".presentation.ability.detail.AbilityDetailActivity">

<include
android:id="@+id/item_ability_detail"
layout="@layout/item_ability_description"
<com.google.android.material.appbar.AppBarLayout
android:id="@+id/app_bar_ability_detail"
android:layout_width="match_parent"
android:layout_height="wrap_content"
app:layout_constraintTop_toBottomOf="@id/toolbar_ability_detail"
bind:ability="@{abilityUiModel}" />
android:background="@color/poke_black"
app:liftOnScrollColor="@color/poke_black">

<com.google.android.material.appbar.MaterialToolbar
android:id="@+id/toolbar_ability_detail"
style="@style/CustomToolbarStyle"
android:layout_width="match_parent"
android:layout_height="?attr/actionBarSize"
app:layout_scrollFlags="scroll|enterAlways"
app:title="@string/ability_title_name">

<ImageView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="end|center_vertical"
Copy link
Contributor

Choose a reason for hiding this comment

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

.. 이렇게 쉽게할 수있다니 ㅜㅜ 고생하셨습니다

@murjune murjune changed the title [AN-UI, AN-FEAT] 특성 목록 스크롤 시 앱 바 애니메이션 적용 [AN-UI, AN-FEAT] 특성 디테일 스크롤 시 앱 바 애니메이션 적용 Nov 28, 2024
@murjune murjune changed the title [AN-UI, AN-FEAT] 특성 디테일 스크롤 시 앱 바 애니메이션 적용 [1.2.0/AN-UI, AN-FEAT] 특성 디테일 스크롤 시 앱 바 애니메이션 적용 Dec 1, 2024
Copy link
Contributor

@JoYehyun99 JoYehyun99 left a comment

Choose a reason for hiding this comment

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

👍👍👍👍👍👍👍👍👍👍

Copy link
Contributor

@kkosang kkosang left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 심지 !! 👍

app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@+id/item_ability_detail"
app:layout_behavior="@string/appbar_scrolling_view_behavior"
Copy link
Contributor

Choose a reason for hiding this comment

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

behavior 굿입니다 !! 👍👍


</com.google.android.material.appbar.MaterialToolbar>
<androidx.coordinatorlayout.widget.CoordinatorLayout
Copy link
Contributor

Choose a reason for hiding this comment

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

굿👍👍

Copy link
Contributor

@kkosang kkosang Dec 5, 2024

Choose a reason for hiding this comment

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

가로모드까지 챙기는 꼼꼼함까지 🫢

</layout>


<!--<androidx.constraintlayout.widget.ConstraintLayout-->
Copy link
Contributor

Choose a reason for hiding this comment

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

여기를 주석으로 남겨놓은 이유가 있나요 ?!

@sh1mj1 sh1mj1 merged commit f9d4cc2 into an/develop Dec 6, 2024
2 checks passed
@sh1mj1 sh1mj1 deleted the an/feat/ability-detail-top-bar-collapsing branch December 6, 2024 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AN_FEAT ✨ 안드 새로운 기능 AN_UI 🎨 안드 UI 작업 v1.2.0 🏷️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants