Merge "Enforce constrainedWidth in date and icons" into tm-qpr-dev am: b11a67284f
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20459012 Change-Id: If4cc61cfd07f76437f7667b0972f44d1bc9b6b17 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -59,6 +59,7 @@
|
|||||||
<Layout
|
<Layout
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="@dimen/new_qs_header_non_clickable_element_height"
|
android:layout_height="@dimen/new_qs_header_non_clickable_element_height"
|
||||||
|
app:layout_constrainedWidth="true"
|
||||||
app:layout_constraintHeight_min="@dimen/new_qs_header_non_clickable_element_height"
|
app:layout_constraintHeight_min="@dimen/new_qs_header_non_clickable_element_height"
|
||||||
app:layout_constraintStart_toEndOf="@id/date"
|
app:layout_constraintStart_toEndOf="@id/date"
|
||||||
app:layout_constraintEnd_toStartOf="@id/batteryRemainingIcon"
|
app:layout_constraintEnd_toStartOf="@id/batteryRemainingIcon"
|
||||||
|
|||||||
@@ -51,6 +51,8 @@ object CombinedShadeHeadersConstraintManagerImpl : CombinedShadeHeadersConstrain
|
|||||||
connect(R.id.statusIcons, ConstraintSet.START, R.id.date, ConstraintSet.END)
|
connect(R.id.statusIcons, ConstraintSet.START, R.id.date, ConstraintSet.END)
|
||||||
connect(R.id.privacy_container, ConstraintSet.START, R.id.date, ConstraintSet.END)
|
connect(R.id.privacy_container, ConstraintSet.START, R.id.date, ConstraintSet.END)
|
||||||
constrainWidth(R.id.statusIcons, ViewGroup.LayoutParams.WRAP_CONTENT)
|
constrainWidth(R.id.statusIcons, ViewGroup.LayoutParams.WRAP_CONTENT)
|
||||||
|
constrainedWidth(R.id.date, true)
|
||||||
|
constrainedWidth(R.id.statusIcons, true)
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
@@ -92,6 +94,8 @@ object CombinedShadeHeadersConstraintManagerImpl : CombinedShadeHeadersConstrain
|
|||||||
centerEnd,
|
centerEnd,
|
||||||
ConstraintSet.END
|
ConstraintSet.END
|
||||||
)
|
)
|
||||||
|
constrainedWidth(R.id.date, true)
|
||||||
|
constrainedWidth(R.id.statusIcons, true)
|
||||||
},
|
},
|
||||||
qsConstraintsChanges = {
|
qsConstraintsChanges = {
|
||||||
setGuidelineBegin(centerStart, offsetFromEdge)
|
setGuidelineBegin(centerStart, offsetFromEdge)
|
||||||
|
|||||||
@@ -363,6 +363,22 @@ class CombinedShadeHeaderConstraintsTest : SysuiTestCase() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun testEmptyCutoutDateIconsAreConstrainedWidth() {
|
||||||
|
CombinedShadeHeadersConstraintManagerImpl.emptyCutoutConstraints()()
|
||||||
|
|
||||||
|
assertThat(qqsConstraint.getConstraint(R.id.date).layout.constrainedWidth).isTrue()
|
||||||
|
assertThat(qqsConstraint.getConstraint(R.id.statusIcons).layout.constrainedWidth).isTrue()
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun testCenterCutoutDateIconsAreConstrainedWidth() {
|
||||||
|
CombinedShadeHeadersConstraintManagerImpl.centerCutoutConstraints(false, 10)()
|
||||||
|
|
||||||
|
assertThat(qqsConstraint.getConstraint(R.id.date).layout.constrainedWidth).isTrue()
|
||||||
|
assertThat(qqsConstraint.getConstraint(R.id.statusIcons).layout.constrainedWidth).isTrue()
|
||||||
|
}
|
||||||
|
|
||||||
private operator fun ConstraintsChanges.invoke() {
|
private operator fun ConstraintsChanges.invoke() {
|
||||||
qqsConstraintsChanges?.invoke(qqsConstraint)
|
qqsConstraintsChanges?.invoke(qqsConstraint)
|
||||||
qsConstraintsChanges?.invoke(qsConstraint)
|
qsConstraintsChanges?.invoke(qsConstraint)
|
||||||
|
|||||||
Reference in New Issue
Block a user