aixan/lawnchair
1
0
Fork 0
mirror of https://github.com/LawnchairLauncher/lawnchair.git synced 2026-02-19 02:38:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Fix hotseat border space crash GSI" into udc-qpr-dev

Browse Source
This commit is contained in:
Jordan Silva
2023-08-23 18:11:03 +00:00
committed by Android (Google) Code Review
parent 7e1c5d0b74 d92271849c
commit c6b6a4f3d5
2 changed files with 25 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/com/android/launcher3/DeviceProfile.java
View File

@@ -1134,10 +1134,11 @@ public class DeviceProfile {
* This method calculates the space between the icons to achieve a certain width.
*/
private int calculateHotseatBorderSpace(float hotseatWidthPx, int numExtraBorder) {
int numBorders = (numShownHotseatIcons - 1 + numExtraBorder);
if (numBorders <= 0) return 0;
float hotseatIconsTotalPx = iconSizePx * numShownHotseatIcons;
int hotseatBorderSpacePx =
(int) (hotseatWidthPx - hotseatIconsTotalPx)
/ (numShownHotseatIcons - 1 + numExtraBorder);
int hotseatBorderSpacePx = (int) (hotseatWidthPx - hotseatIconsTotalPx) / numBorders;
return Math.min(hotseatBorderSpacePx, mMaxHotseatIconSpacePx);
}

21
tests/src/com/android/launcher3/nonquickstep/HotseatWidthCalculationTest.kt
View File

@@ -158,4 +158,25 @@ class HotseatWidthCalculationTest : FakeInvariantDeviceProfileTest() {
assertThat(dp.isQsbInline).isFalse()
assertThat(dp.hotseatQsbWidth).isEqualTo(1095)
}
@Test
fun border_space_should_be_zero_when_numHotseatIcons_is_smallerOrEqual_1() {
initializeVarsForTablet(isGestureMode = false)
windowBounds = WindowBounds(Rect(0, 0, 1800, 2560), Rect(0, 104, 0, 0))
val numShownHotseatIcons = listOf(-1, 0, 1)
for (numHotseatIcons in numShownHotseatIcons) {
inv?.numShownHotseatIcons = numHotseatIcons
val dp = newDP()
dp.isTaskbarPresentInApps = true
assertThat(dp.numShownHotseatIcons).isEqualTo(numHotseatIcons)
assertThat(dp.hotseatBorderSpace).isEqualTo(0)
assertThat(dp.getHotseatLayoutPadding(context).left).isEqualTo(177)
assertThat(dp.getHotseatLayoutPadding(context).right).isEqualTo(177)
assertThat(dp.hotseatQsbWidth).isEqualTo(1445)
}
}
}