Skip to content

Commit f3801d9

Browse files
cpojergrabbou
authored andcommitted
Revert "improve RTL (#24069)" (#24580)
Summary: This reverts commit b3c7496. Fixes #24267 [Android] [Fixed] - Invalid text alignment for RTL fonts. Pull Request resolved: #24580 Differential Revision: D15061667 Pulled By: cpojer fbshipit-source-id: 6d02c9e938f1f8630ba691f57bdf79fd57db3bb2
1 parent 2b7d799 commit f3801d9

File tree

4 files changed

+23
-24
lines changed

4 files changed

+23
-24
lines changed

ReactAndroid/src/main/java/com/facebook/react/views/text/ReactBaseTextShadowNode.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -315,10 +315,10 @@ public ReactBaseTextShadowNode() {
315315
private int getTextAlign() {
316316
int textAlign = mTextAlign;
317317
if (getLayoutDirection() == YogaDirection.RTL) {
318-
if (textAlign == Gravity.END) {
319-
textAlign = Gravity.START;
320-
} else if (textAlign == Gravity.START) {
321-
textAlign = Gravity.END;
318+
if (textAlign == Gravity.RIGHT) {
319+
textAlign = Gravity.LEFT;
320+
} else if (textAlign == Gravity.LEFT) {
321+
textAlign = Gravity.RIGHT;
322322
}
323323
}
324324
return textAlign;
@@ -364,7 +364,7 @@ public void setTextAlign(@Nullable String textAlign) {
364364
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
365365
mJustificationMode = Layout.JUSTIFICATION_MODE_INTER_WORD;
366366
}
367-
mTextAlign = Gravity.START;
367+
mTextAlign = Gravity.LEFT;
368368
} else {
369369
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
370370
mJustificationMode = Layout.JUSTIFICATION_MODE_NONE;
@@ -373,9 +373,9 @@ public void setTextAlign(@Nullable String textAlign) {
373373
if (textAlign == null || "auto".equals(textAlign)) {
374374
mTextAlign = Gravity.NO_GRAVITY;
375375
} else if ("left".equals(textAlign)) {
376-
mTextAlign = Gravity.START;
376+
mTextAlign = Gravity.LEFT;
377377
} else if ("right".equals(textAlign)) {
378-
mTextAlign = Gravity.END;
378+
mTextAlign = Gravity.RIGHT;
379379
} else if ("center".equals(textAlign)) {
380380
mTextAlign = Gravity.CENTER_HORIZONTAL;
381381
} else {

ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextShadowNode.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,10 @@ public long measure(
7676

7777
Layout.Alignment alignment = Layout.Alignment.ALIGN_NORMAL;
7878
switch (getTextAlign()) {
79-
case Gravity.START:
79+
case Gravity.LEFT:
8080
alignment = Layout.Alignment.ALIGN_NORMAL;
8181
break;
82-
case Gravity.END:
82+
case Gravity.RIGHT:
8383
alignment = Layout.Alignment.ALIGN_OPPOSITE;
8484
break;
8585
case Gravity.CENTER_HORIZONTAL:
@@ -179,10 +179,10 @@ private void initMeasureFunction() {
179179
private int getTextAlign() {
180180
int textAlign = mTextAlign;
181181
if (getLayoutDirection() == YogaDirection.RTL) {
182-
if (textAlign == Gravity.END) {
183-
textAlign = Gravity.START;
184-
} else if (textAlign == Gravity.START) {
185-
textAlign = Gravity.END;
182+
if (textAlign == Gravity.RIGHT) {
183+
textAlign = Gravity.LEFT;
184+
} else if (textAlign == Gravity.LEFT) {
185+
textAlign = Gravity.RIGHT;
186186
}
187187
}
188188
return textAlign;

ReactAndroid/src/main/java/com/facebook/react/views/text/TextAttributeProps.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -164,10 +164,10 @@ public float getEffectiveLineHeight() {
164164
public int getTextAlign() {
165165
int textAlign = mTextAlign;
166166
if (getLayoutDirection() == YogaDirection.RTL) {
167-
if (textAlign == Gravity.END) {
168-
textAlign = Gravity.START;
169-
} else if (textAlign == Gravity.START) {
170-
textAlign = Gravity.END;
167+
if (textAlign == Gravity.RIGHT) {
168+
textAlign = Gravity.LEFT;
169+
} else if (textAlign == Gravity.LEFT) {
170+
textAlign = Gravity.RIGHT;
171171
}
172172
}
173173
return textAlign;
@@ -210,7 +210,7 @@ public void setTextAlign(@Nullable String textAlign) {
210210
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
211211
mJustificationMode = Layout.JUSTIFICATION_MODE_INTER_WORD;
212212
}
213-
mTextAlign = Gravity.START;
213+
mTextAlign = Gravity.LEFT;
214214
} else {
215215
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
216216
mJustificationMode = Layout.JUSTIFICATION_MODE_NONE;
@@ -219,9 +219,9 @@ public void setTextAlign(@Nullable String textAlign) {
219219
if (textAlign == null || "auto".equals(textAlign)) {
220220
mTextAlign = Gravity.NO_GRAVITY;
221221
} else if ("left".equals(textAlign)) {
222-
mTextAlign = Gravity.START;
222+
mTextAlign = Gravity.LEFT;
223223
} else if ("right".equals(textAlign)) {
224-
mTextAlign = Gravity.END;
224+
mTextAlign = Gravity.RIGHT;
225225
} else if ("center".equals(textAlign)) {
226226
mTextAlign = Gravity.CENTER_HORIZONTAL;
227227
} else {

ReactAndroid/src/main/java/com/facebook/react/views/textinput/ReactTextInputManager.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99

1010
import static android.view.View.FOCUS_FORWARD;
1111

12-
import android.annotation.TargetApi;
1312
import android.graphics.PorterDuff;
1413
import android.graphics.Typeface;
1514
import android.graphics.drawable.Drawable;
@@ -473,7 +472,7 @@ public void setTextAlign(ReactEditText view, @Nullable String textAlign) {
473472
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
474473
view.setJustificationMode(Layout.JUSTIFICATION_MODE_INTER_WORD);
475474
}
476-
view.setGravityHorizontal(Gravity.START);
475+
view.setGravityHorizontal(Gravity.LEFT);
477476
} else {
478477
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
479478
view.setJustificationMode(Layout.JUSTIFICATION_MODE_NONE);
@@ -482,9 +481,9 @@ public void setTextAlign(ReactEditText view, @Nullable String textAlign) {
482481
if (textAlign == null || "auto".equals(textAlign)) {
483482
view.setGravityHorizontal(Gravity.NO_GRAVITY);
484483
} else if ("left".equals(textAlign)) {
485-
view.setGravityHorizontal(Gravity.START);
484+
view.setGravityHorizontal(Gravity.LEFT);
486485
} else if ("right".equals(textAlign)) {
487-
view.setGravityHorizontal(Gravity.END);
486+
view.setGravityHorizontal(Gravity.RIGHT);
488487
} else if ("center".equals(textAlign)) {
489488
view.setGravityHorizontal(Gravity.CENTER_HORIZONTAL);
490489
} else {

0 commit comments

Comments
 (0)