Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Revert "Adding padding to display the full clock for Burmese language"" into udc-qpr-dev am: eaf5ec33ca

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/24154810

Change-Id: I9c50f5690ba92666641199e1347b6054531a1e00
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
William Leshner
2023-07-21 00:24:47 +00:00
committed by Automerger Merge Worker
parent 0c4208d943 eaf5ec33ca
commit f9751c030c
4 changed files with 14 additions and 184 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
packages/SystemUI/shared/res/values-my/bools.xml
View File

@@ -1,25 +0,0 @@
<?xml version="1.0" encoding="utf-8"?>
<!-- Copyright (C) 2023 The Android Open Source Project
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
-->
<!-- Formatting note: terminate all comments with a period, to avoid breaking
the documentation output. To suppress comment lines from the documentation
output, insert an eat-comment element after the comment lines.
-->
<resources>
<!-- Whether to add padding at the bottom of the complication clock -->
<bool name="dream_overlay_complication_clock_bottom_padding">true</bool>
</resources>

25
packages/SystemUI/shared/res/values/bools.xml
View File

@@ -1,25 +0,0 @@
<?xml version="1.0" encoding="utf-8"?>
<!-- Copyright (C) 2023 The Android Open Source Project
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
-->
<!-- Formatting note: terminate all comments with a period, to avoid breaking
the documentation output. To suppress comment lines from the documentation
output, insert an eat-comment element after the comment lines.
-->
<resources>
<!-- Whether to add padding at the bottom of the complication clock -->
<bool name="dream_overlay_complication_clock_bottom_padding">false</bool>
</resources>

56
packages/SystemUI/shared/src/com/android/systemui/shared/shadow/DoubleShadowTextClock.kt
View File

@@ -16,51 +16,34 @@
package com.android.systemui.shared.shadow
import android.content.Context
import android.content.res.Resources
import android.content.res.TypedArray
import android.graphics.Canvas
import android.util.AttributeSet
import android.widget.TextClock
import com.android.systemui.shared.R
import com.android.systemui.shared.shadow.DoubleShadowTextHelper.ShadowInfo
import com.android.systemui.shared.shadow.DoubleShadowTextHelper.applyShadows
import javax.inject.Inject
import kotlin.math.floor
/** Extension of [TextClock] which draws two shadows on the text (ambient and key shadows) */
class ResourcesProvider @Inject constructor(private val context: Context) {
fun getResources(): Resources {
return context.resources
}
fun getBoolean(resourceId: Int): Boolean {
return getResources().getBoolean(resourceId)
}
}
class DoubleShadowTextClock
@JvmOverloads
constructor(
private val resourcesProvider: ResourcesProvider,
context: Context,
attrs: AttributeSet? = null,
defStyleAttr: Int = 0,
defStyleRes: Int = 0,
attributesInput: TypedArray? = null
defStyleRes: Int = 0
) : TextClock(context, attrs, defStyleAttr, defStyleRes) {
private val mAmbientShadowInfo: ShadowInfo
private val mKeyShadowInfo: ShadowInfo
init {
var attributes: TypedArray =
attributesInput
?: context.obtainStyledAttributes(
attrs,
R.styleable.DoubleShadowTextClock,
defStyleAttr,
defStyleRes
)
val attributes =
context.obtainStyledAttributes(
attrs,
R.styleable.DoubleShadowTextClock,
defStyleAttr,
defStyleRes
)
try {
val keyShadowBlur =
attributes.getDimensionPixelSize(R.styleable.DoubleShadowTextClock_keyShadowBlur, 0)
@@ -115,29 +98,18 @@ constructor(
0
)
if (removeTextDescent) {
val addBottomPaddingToClock =
resourcesProvider.getBoolean(
R.bool.dream_overlay_complication_clock_bottom_padding
)
val metrics = paint.fontMetrics
val padding =
if (addBottomPaddingToClock) {
textDescentExtraPadding +
floor(metrics.descent.toDouble()).toInt() / paddingDividedOffset
} else {
textDescentExtraPadding - floor(metrics.descent.toDouble()).toInt()
}
setPaddingRelative(0, 0, 0, padding)
setPaddingRelative(
0,
0,
0,
textDescentExtraPadding - floor(paint.fontMetrics.descent.toDouble()).toInt()
)
}
} finally {
attributes.recycle()
}
}
companion object {
private val paddingDividedOffset = 2
}
public override fun onDraw(canvas: Canvas) {
applyShadows(mKeyShadowInfo, mAmbientShadowInfo, this, canvas) { super.onDraw(canvas) }
}

92
packages/SystemUI/tests/src/com/android/systemui/shadow/DoubleShadowTextClockTest.kt
View File

@@ -1,92 +0,0 @@
/*
* Copyright (C) 2023 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License
*/
package com.android.systemui.shadow
import android.content.Context
import android.content.res.TypedArray
import android.testing.AndroidTestingRunner
import android.util.AttributeSet
import androidx.test.filters.SmallTest
import com.android.systemui.R
import com.android.systemui.SysuiTestCase
import com.android.systemui.shared.shadow.DoubleShadowTextClock
import com.android.systemui.shared.shadow.ResourcesProvider
import com.android.systemui.util.mockito.whenever
import junit.framework.Assert.assertTrue
import org.junit.Before
import org.junit.Rule
import org.junit.Test
import org.junit.runner.RunWith
import org.mockito.Mock
import org.mockito.MockitoAnnotations
import org.mockito.junit.MockitoJUnit
import org.mockito.junit.MockitoRule
@SmallTest
@RunWith(AndroidTestingRunner::class)
class DoubleShadowTextClockTest : SysuiTestCase() {
@get:Rule val mockito: MockitoRule = MockitoJUnit.rule()
@Mock lateinit var resourcesProvider: ResourcesProvider
@Mock lateinit var attributes: TypedArray
private lateinit var context: Context
private var attrs: AttributeSet? = null
@Before
fun setup() {
MockitoAnnotations.initMocks(this)
context = getContext()
whenever(attributes.getBoolean(R.styleable.DoubleShadowTextClock_removeTextDescent, false))
.thenReturn(true)
}
@Test
fun testAddingPaddingToBottomOfClockWhenConfigIsTrue() {
whenever(
resourcesProvider.getBoolean(R.bool.dream_overlay_complication_clock_bottom_padding)
)
.thenReturn(true)
val doubleShadowTextClock =
DoubleShadowTextClock(
resourcesProvider = resourcesProvider,
context = context,
attrs = attrs,
attributesInput = attributes
)
assertTrue(doubleShadowTextClock.paddingBottom > 0)
}
@Test
fun testRemovingPaddingToBottomOfClockWhenConfigIsFalse() {
whenever(
resourcesProvider.getBoolean(R.bool.dream_overlay_complication_clock_bottom_padding)
)
.thenReturn(false)
val doubleShadowTextClock =
DoubleShadowTextClock(
resourcesProvider = resourcesProvider,
context = context,
attrs = attrs,
attributesInput = attributes
)
assertTrue(doubleShadowTextClock.paddingBottom < 0)
}
}