Skip to content

Commit 46fde74

Browse files
imhappidsn5ft
authored andcommitted
[Theme] Added missing Shape Attributes to Dialog themes
PiperOrigin-RevId: 789801786 (cherry picked from commit 7fe9ef9)
1 parent 8f58fcc commit 46fde74

File tree

2 files changed

+47
-0
lines changed

2 files changed

+47
-0
lines changed

lib/java/com/google/android/material/dialog/res/values/themes_base.xml

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -255,11 +255,22 @@
255255

256256
<!-- Shape attributes. -->
257257
<item name="shapeCornerFamily">@integer/m3_sys_shape_corner_full_corner_family</item>
258+
<item name="shapeCornerSizeExtraSmall">@dimen/m3_sys_shape_corner_value_extra_small</item>
259+
<item name="shapeCornerSizeSmall">@dimen/m3_sys_shape_corner_value_small</item>
260+
<item name="shapeCornerSizeMedium">@dimen/m3_sys_shape_corner_value_medium</item>
261+
<item name="shapeCornerSizeLarge">@dimen/m3_sys_shape_corner_value_large</item>
262+
<item name="shapeCornerSizeLargeIncreased">@dimen/m3_sys_shape_corner_value_large_increased</item>
263+
<item name="shapeCornerSizeExtraLarge">@dimen/m3_sys_shape_corner_value_extra_large</item>
264+
<item name="shapeCornerSizeExtraLargeIncreased">@dimen/m3_sys_shape_corner_value_extra_large_increased</item>
265+
<item name="shapeCornerSizeExtraExtraLarge">@dimen/m3_sys_shape_corner_value_extra_extra_large</item>
258266
<item name="shapeAppearanceCornerExtraSmall">@style/ShapeAppearance.Material3.Corner.ExtraSmall</item>
259267
<item name="shapeAppearanceCornerSmall">@style/ShapeAppearance.Material3.Corner.Small</item>
260268
<item name="shapeAppearanceCornerMedium">@style/ShapeAppearance.Material3.Corner.Medium</item>
261269
<item name="shapeAppearanceCornerLarge">@style/ShapeAppearance.Material3.Corner.Large</item>
270+
<item name="shapeAppearanceCornerLargeIncreased">@style/ShapeAppearance.Material3.Corner.LargeIncreased</item>
262271
<item name="shapeAppearanceCornerExtraLarge">@style/ShapeAppearance.Material3.Corner.ExtraLarge</item>
272+
<item name="shapeAppearanceCornerExtraLargeIncreased">@style/ShapeAppearance.Material3.Corner.ExtraLargeIncreased</item>
273+
<item name="shapeAppearanceCornerExtraExtraLarge">@style/ShapeAppearance.Material3.Corner.ExtraExtraLarge</item>
263274

264275
<!-- Deprecated shape styles. -->
265276
<item name="shapeAppearanceSmallComponent">@style/ShapeAppearance.Material3.SmallComponent</item>
@@ -556,11 +567,22 @@
556567

557568
<!-- Shape attributes. -->
558569
<item name="shapeCornerFamily">@integer/m3_sys_shape_corner_full_corner_family</item>
570+
<item name="shapeCornerSizeExtraSmall">@dimen/m3_sys_shape_corner_value_extra_small</item>
571+
<item name="shapeCornerSizeSmall">@dimen/m3_sys_shape_corner_value_small</item>
572+
<item name="shapeCornerSizeMedium">@dimen/m3_sys_shape_corner_value_medium</item>
573+
<item name="shapeCornerSizeLarge">@dimen/m3_sys_shape_corner_value_large</item>
574+
<item name="shapeCornerSizeLargeIncreased">@dimen/m3_sys_shape_corner_value_large_increased</item>
575+
<item name="shapeCornerSizeExtraLarge">@dimen/m3_sys_shape_corner_value_extra_large</item>
576+
<item name="shapeCornerSizeExtraLargeIncreased">@dimen/m3_sys_shape_corner_value_extra_large_increased</item>
577+
<item name="shapeCornerSizeExtraExtraLarge">@dimen/m3_sys_shape_corner_value_extra_extra_large</item>
559578
<item name="shapeAppearanceCornerExtraSmall">@style/ShapeAppearance.Material3.Corner.ExtraSmall</item>
560579
<item name="shapeAppearanceCornerSmall">@style/ShapeAppearance.Material3.Corner.Small</item>
561580
<item name="shapeAppearanceCornerMedium">@style/ShapeAppearance.Material3.Corner.Medium</item>
562581
<item name="shapeAppearanceCornerLarge">@style/ShapeAppearance.Material3.Corner.Large</item>
582+
<item name="shapeAppearanceCornerLargeIncreased">@style/ShapeAppearance.Material3.Corner.LargeIncreased</item>
563583
<item name="shapeAppearanceCornerExtraLarge">@style/ShapeAppearance.Material3.Corner.ExtraLarge</item>
584+
<item name="shapeAppearanceCornerExtraLargeIncreased">@style/ShapeAppearance.Material3.Corner.ExtraLargeIncreased</item>
585+
<item name="shapeAppearanceCornerExtraExtraLarge">@style/ShapeAppearance.Material3.Corner.ExtraExtraLarge</item>
564586

565587
<!-- Deprecated shape styles. -->
566588
<item name="shapeAppearanceSmallComponent">@style/ShapeAppearance.Material3.SmallComponent</item>

lib/javatests/com/google/android/material/theme/ThemeTest.java

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -476,6 +476,26 @@ public class ThemeTest {
476476
R.attr.toolbarStyle,
477477
R.attr.toolbarSurfaceStyle);
478478

479+
/** These are shape styles that should be the same for *all* M3 full themes. */
480+
private static final ImmutableList<Integer> M3_SHAPE_ATTRIBUTES =
481+
ImmutableList.of(
482+
R.attr.shapeCornerFamily,
483+
R.attr.shapeCornerSizeExtraSmall,
484+
R.attr.shapeCornerSizeSmall,
485+
R.attr.shapeCornerSizeMedium,
486+
R.attr.shapeCornerSizeLarge,
487+
R.attr.shapeCornerSizeLargeIncreased,
488+
R.attr.shapeCornerSizeExtraExtraLarge,
489+
R.attr.shapeCornerSizeExtraLargeIncreased,
490+
R.attr.shapeCornerSizeExtraExtraLarge,
491+
R.attr.shapeAppearanceCornerExtraSmall,
492+
R.attr.shapeAppearanceCornerSmall,
493+
R.attr.shapeAppearanceCornerMedium,
494+
R.attr.shapeAppearanceCornerLarge,
495+
R.attr.shapeAppearanceCornerLargeIncreased,
496+
R.attr.shapeAppearanceCornerExtraLarge,
497+
R.attr.shapeAppearanceCornerExtraLargeIncreased);
498+
479499
/**
480500
* These are all the attributes where full themes should match {@code Theme.Material3.Light} or
481501
* {@code Theme_Material3_Dark}.
@@ -486,6 +506,7 @@ public class ThemeTest {
486506
.addAll(M3_FULL_TYPOGRAPHY_ATTRIBUTES)
487507
.addAll(M3_FULL_MOTION_ATTRIBUTES)
488508
.addAll(M3_COMMON_WIDGET_STYLE_ATTRIBUTES)
509+
.addAll(M3_SHAPE_ATTRIBUTES)
489510
.build();
490511

491512
@Parameters(name = "{0}")
@@ -615,6 +636,10 @@ public static ImmutableList<Object[]> getTestData() {
615636
R.style.Theme_Material3_Dark,
616637
R.style.Theme_Material3_Light,
617638
M3_FULL_MOTION_ATTRIBUTES))
639+
// Compare M3 Light and Dark themes - they should have the same shape styles.
640+
.addAll(
641+
createTestData(
642+
R.style.Theme_Material3_Dark, R.style.Theme_Material3_Light, M3_SHAPE_ATTRIBUTES))
618643
.build();
619644
}
620645

0 commit comments

Comments
 (0)