Skip to content

Commit 54d62f3

Browse files
authored
Make profile app bar icons part of the scrollable content (#3138)
1 parent ca2fd95 commit 54d62f3

File tree

3 files changed

+33
-38
lines changed

3 files changed

+33
-38
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
* Bug Fixes
1717
* Keep the playing service in the foreground when casting
1818
([#3094](https://github.com/Automattic/pocket-casts-android/pull/3094))
19+
* Scroll profile tab app bar icons with the content in the landscape orientation
20+
([#3138](https://github.com/Automattic/pocket-casts-android/pull/3138))
1921
* Updates
2022
* Move Up Next clear queue button to app bar
2123
([#3068](https://github.com/Automattic/pocket-casts-android/pull/3068))

modules/features/profile/src/main/res/layout-land/fragment_profile.xml

Lines changed: 24 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -171,39 +171,32 @@
171171
app:layout_constraintStart_toStartOf="parent"
172172
app:layout_constraintTop_toBottomOf="@+id/btnRefresh"
173173
app:layout_constraintBottom_toBottomOf="parent"/>
174+
175+
<androidx.compose.ui.platform.ComposeView
176+
android:id="@+id/btnGift"
177+
android:layout_width="44dp"
178+
android:layout_height="44dp"
179+
android:layout_gravity="end"
180+
android:layout_marginTop="8dp"
181+
android:layout_marginEnd="8dp"
182+
app:layout_constraintEnd_toStartOf="@id/btnSettings"
183+
app:layout_constraintTop_toTopOf="parent" />
184+
185+
<ImageButton
186+
android:id="@+id/btnSettings"
187+
android:layout_width="44dp"
188+
android:layout_height="44dp"
189+
android:layout_marginTop="8dp"
190+
android:layout_marginEnd="8dp"
191+
android:background="?android:attr/actionBarItemBackground"
192+
android:contentDescription="@string/settings"
193+
android:src="@drawable/ic_profile_settings"
194+
app:layout_constraintEnd_toEndOf="parent"
195+
app:layout_constraintTop_toTopOf="parent"
196+
app:tint="?attr/primary_icon_01" />
197+
174198
</androidx.constraintlayout.widget.ConstraintLayout>
175199

176200
</androidx.core.widget.NestedScrollView>
177201

178-
<androidx.constraintlayout.widget.ConstraintLayout
179-
android:layout_width="wrap_content"
180-
android:layout_height="wrap_content"
181-
android:layout_gravity="end"
182-
android:layout_marginTop="7dp"
183-
android:layout_marginEnd="6dp"
184-
android:layout_marginBottom="10dp">
185-
186-
<androidx.compose.ui.platform.ComposeView
187-
android:id="@+id/btnGift"
188-
android:layout_width="wrap_content"
189-
android:layout_height="wrap_content"
190-
android:layout_gravity="end"
191-
android:layout_marginEnd="8dp"
192-
app:layout_constraintEnd_toStartOf="@id/btnSettings"
193-
app:layout_constraintTop_toTopOf="parent"/>
194-
195-
<ImageButton
196-
android:id="@+id/btnSettings"
197-
android:layout_width="44dp"
198-
android:layout_height="44dp"
199-
android:layout_marginBottom="10dp"
200-
android:background="?android:attr/actionBarItemBackground"
201-
android:contentDescription="@string/settings"
202-
android:src="@drawable/ic_profile_settings"
203-
app:layout_constraintEnd_toEndOf="parent"
204-
app:layout_constraintTop_toTopOf="parent"
205-
app:tint="?attr/primary_icon_01" />
206-
207-
</androidx.constraintlayout.widget.ConstraintLayout>
208-
209202
</FrameLayout>

modules/features/profile/src/main/res/layout/fragment_profile.xml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -172,19 +172,19 @@
172172

173173
<androidx.compose.ui.platform.ComposeView
174174
android:id="@+id/btnGift"
175-
android:layout_width="wrap_content"
176-
android:layout_height="wrap_content"
177-
android:layout_marginTop="7dp"
178-
android:layout_marginStart="7dp"
175+
android:layout_width="44dp"
176+
android:layout_height="44dp"
177+
android:layout_marginStart="8dp"
178+
android:layout_marginTop="8dp"
179179
app:layout_constraintStart_toStartOf="parent"
180-
app:layout_constraintTop_toTopOf="parent"/>
180+
app:layout_constraintTop_toTopOf="parent" />
181181

182182
<ImageButton
183183
android:id="@+id/btnSettings"
184184
android:layout_width="44dp"
185185
android:layout_height="44dp"
186-
android:layout_marginTop="7dp"
187-
android:layout_marginEnd="7dp"
186+
android:layout_marginTop="8dp"
187+
android:layout_marginEnd="8dp"
188188
android:background="?android:attr/actionBarItemBackground"
189189
android:contentDescription="@string/settings"
190190
android:src="@drawable/ic_profile_settings"

0 commit comments

Comments
 (0)