Skip to content

Commit 36c6e4a

Browse files
committed
Refactor: Move clearAndSetSemantics modifier to LoopingText
1 parent 777cb76 commit 36c6e4a

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

WordPress/src/jetpack/java/org/wordpress/android/ui/accounts/login/components/LoopingTextWithBackground.kt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import androidx.compose.ui.draw.paint
1111
import androidx.compose.ui.layout.ContentScale
1212
import androidx.compose.ui.res.colorResource
1313
import androidx.compose.ui.res.painterResource
14+
import androidx.compose.ui.semantics.clearAndSetSemantics
1415
import androidx.compose.ui.tooling.preview.Devices
1516
import androidx.compose.ui.tooling.preview.Preview
1617
import androidx.compose.ui.unit.dp
@@ -33,6 +34,7 @@ fun LoopingTextWithBackground(
3334
) {
3435
LoopingText(
3536
modifier = Modifier
37+
.clearAndSetSemantics { }
3638
.fillMaxSize()
3739
.padding(horizontal = 20.dp)
3840
.then(textModifier)

WordPress/src/jetpack/java/org/wordpress/android/ui/accounts/login/components/RepeatingColumn.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import androidx.compose.foundation.layout.Column
44
import androidx.compose.runtime.Composable
55
import androidx.compose.ui.Modifier
66
import androidx.compose.ui.layout.Layout
7-
import androidx.compose.ui.semantics.clearAndSetSemantics
87
import androidx.compose.ui.unit.Constraints
98

109
/** This composable accepts a child composable, and repeats that composable `repeat` times (default 3). It places the
@@ -25,7 +24,7 @@ fun RepeatingColumn(
2524
) {
2625
Layout(
2726
content = {
28-
Column(modifier.clearAndSetSemantics {}) {
27+
Column(modifier = modifier) {
2928
repeat(repeat) {
3029
content()
3130
}

0 commit comments

Comments
 (0)