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] 특성 목록 스크롤 시 앱 바 애니메이션 적용 #464

Merged
merged 17 commits into from
Dec 6, 2024

Conversation

sh1mj1
Copy link
Contributor

@sh1mj1 sh1mj1 commented Nov 27, 2024

작업 영상

Untitled.Video.mov

작업한 내용

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

PR 포인트

🚀Next Feature

  • 특성 상세에서 스크롤 시 애니메이션

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.

고생하셨습니다 심지! string 지역화가 아직 머지가 안돼서 같이 뭉태기로 보이는게 맞겠죵?

Comment on lines 13 to 40
<androidx.coordinatorlayout.widget.CoordinatorLayout
android:layout_width="match_parent"
android:layout_height="match_parent">

<com.google.android.material.appbar.MaterialToolbar
android:id="@+id/toolbar_ability"
style="@style/CustomToolbarStyle"
android:layout_width="0dp"
android:layout_height="?attr/actionBarSize"
android:contentInsetStartWithNavigation="0dp"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:title="@string/ability_title_name">

<androidx.appcompat.widget.SearchView
style="@style/CustomSearchViewStyle"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="end|center_vertical"
android:theme="@style/CustomSearchViewTheme"
app:onQueryTextChange="@{vm}"
app:queryHint="@string/ability_search_hint" />

</com.google.android.material.appbar.MaterialToolbar>
android:layout_height="match_parent"
tools:context=".presentation.ability.AbilityActivity">

<com.google.android.material.appbar.AppBarLayout
android:id="@+id/app_bar_ability"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:background="@color/poke_black"
app:liftOnScrollColor="@color/poke_black">

<com.google.android.material.appbar.MaterialToolbar
android:id="@+id/toolbar_ability"
style="@style/CustomToolbarStyle"
android:layout_width="match_parent"
android:layout_height="match_parent"
app:layout_scrollFlags="scroll|enterAlways"
app:title="@string/ability_title_name"
app:titleTextColor="@color/poke_white">

<androidx.appcompat.widget.SearchView
style="@style/CustomSearchViewStyle"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="end|center_vertical"
app:onQueryTextChange="@{vm}"
app:queryHint="@string/ability_search_hint" />
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] 특성 목록 스크롤 시 앱 바 애니메이션 적용 [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.

👍👍👍👍👍

@sh1mj1 sh1mj1 force-pushed the an/feat/ability-top-bar-collapsing branch from 6ce9471 to 4f031b0 Compare December 5, 2024 08:33
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.

리뷰가 늦었네요 미안합니다 😱

@@ -10,51 +10,48 @@
type="poke.rogue.helper.presentation.ability.AbilityViewModel" />
</data>


<androidx.constraintlayout.widget.ConstraintLayout
<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.

굿입니다 👍👍

@sh1mj1 sh1mj1 merged commit 24dbea9 into an/develop Dec 6, 2024
2 checks passed
@sh1mj1 sh1mj1 deleted the an/feat/ability-top-bar-collapsing branch December 6, 2024 00:13
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