Skip to content

Commit f2c823b

Browse files
Refactor native ad layouts for consistent padding
1 parent 97575b7 commit f2c823b

File tree

2 files changed

+35
-41
lines changed

2 files changed

+35
-41
lines changed

app/src/main/res/layout/large_home_banner_native_ad.xml

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,19 +11,14 @@
1111
android:layout_height="wrap_content"
1212
app:cardCornerRadius="24dp">
1313

14-
<include
15-
layout="@layout/ad_attribution"
16-
android:layout_margin="16dp"
17-
android:layout_gravity="top|start"
18-
android:paddingHorizontal="8dp"
19-
android:paddingVertical="4dp" />
20-
2114
<LinearLayout
2215
android:layout_width="match_parent"
2316
android:layout_height="wrap_content"
2417
android:layout_margin="16dp"
2518
android:orientation="vertical">
2619

20+
<include layout="@layout/ad_attribution" />
21+
2722
<com.google.android.material.card.MaterialCardView
2823
style="@style/Widget.Material3.CardView.Filled"
2924
android:layout_width="match_parent"

app/src/main/res/layout/small_home_banner_native_ad.xml

Lines changed: 33 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -14,53 +14,52 @@
1414
android:layout_marginBottom="16dp"
1515
app:shapeAppearanceOverlay="@style/ShapeAppearanceOverlay.CardViewBottomRounded">
1616

17-
<include
18-
layout="@layout/ad_attribution"
19-
android:layout_gravity="top|start"
20-
android:layout_margin="8dp" />
21-
2217
<LinearLayout
2318
android:layout_width="match_parent"
2419
android:layout_height="wrap_content"
25-
android:orientation="horizontal"
26-
android:gravity="center_vertical"
20+
android:orientation="vertical"
2721
android:padding="16dp">
2822

29-
<ImageView
30-
android:id="@+id/ad_app_icon"
31-
android:layout_width="40dp"
32-
android:layout_height="40dp"
33-
android:layout_marginEnd="16dp" />
23+
<include layout="@layout/ad_attribution" />
3424

3525
<LinearLayout
36-
android:layout_width="0dp"
26+
android:layout_width="match_parent"
3727
android:layout_height="wrap_content"
38-
android:layout_weight="1"
39-
android:orientation="vertical">
28+
android:layout_marginTop="8dp"
29+
android:gravity="center_vertical"
30+
android:orientation="horizontal">
4031

41-
<TextView
42-
android:id="@+id/ad_headline"
43-
android:layout_width="wrap_content"
32+
<ImageView
33+
android:id="@+id/ad_app_icon"
34+
android:layout_width="40dp"
35+
android:layout_height="40dp"
36+
android:layout_marginEnd="16dp" />
37+
38+
<LinearLayout
39+
android:layout_width="0dp"
4440
android:layout_height="wrap_content"
45-
android:ellipsize="end"
46-
android:maxLines="1"
47-
android:textAppearance="@style/TextAppearance.Material3.TitleMedium" />
41+
android:layout_weight="1"
42+
android:orientation="vertical">
43+
44+
<TextView
45+
android:id="@+id/ad_headline"
46+
android:layout_width="wrap_content"
47+
android:layout_height="wrap_content"
48+
android:textAppearance="@style/TextAppearance.Material3.TitleMedium" />
4849

49-
<TextView
50-
android:id="@+id/ad_body"
50+
<TextView
51+
android:id="@+id/ad_body"
52+
android:layout_width="wrap_content"
53+
android:layout_height="wrap_content"
54+
android:textAppearance="@style/TextAppearance.Material3.BodySmall" />
55+
</LinearLayout>
56+
57+
<com.google.android.material.button.MaterialButton
58+
android:id="@+id/ad_call_to_action"
59+
style="@style/Widget.Material3.Button.TextButton"
5160
android:layout_width="wrap_content"
52-
android:layout_height="wrap_content"
53-
android:ellipsize="end"
54-
android:maxLines="1"
55-
android:textAppearance="@style/TextAppearance.Material3.BodySmall" />
61+
android:layout_height="wrap_content" />
5662
</LinearLayout>
57-
58-
<com.google.android.material.button.MaterialButton
59-
android:id="@+id/ad_call_to_action"
60-
style="@style/Widget.Material3.Button.TextButton"
61-
android:layout_width="wrap_content"
62-
android:layout_height="wrap_content"
63-
android:layout_marginStart="8dp" />
6463
</LinearLayout>
6564
</com.google.android.material.card.MaterialCardView>
6665
</com.google.android.gms.ads.nativead.NativeAdView>

0 commit comments

Comments
 (0)