chinwag/chinwag-android
chinwag/chinwag-android
2
0
Fork 0
Code Issues Pull requests Releases Activity

By explicit about the firstStrong text direction on UGC (#3328)

Browse source 
`activity_account` sets the root text direction to `anyRtl`.

This is OK for UI elements, but can be a problem for user generated content
(UGC) that may contain bidirectional text.

Fix this by explicitly restoring the default behaviour, `firstStrong`, on
views in `activity_account` that can show UGC.

Fixes https://github.com/tuskyapp/Tusky/issues/3294
This commit is contained in:
Nik Clayton 2023-03-02 18:30:26 +01:00 committed by GitHub
parent a792d2c0d6
commit 7261fa0b18
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/src/main/res/layout/activity_account.xml
View file

@ -201,7 +201,8 @@
<com.google.android.material.textfield.TextInputEditText
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:hint="@string/account_note_hint" />
android:hint="@string/account_note_hint"
android:textDirection="firstStrong" />
</com.google.android.material.textfield.TextInputLayout>
@ -225,6 +226,7 @@
android:textColor="?android:textColorTertiary"
android:textIsSelectable="true"
android:textSize="?attr/status_text_medium"
android:textDirection="firstStrong"
app:layout_constraintTop_toBottomOf="@id/saveNoteInfo"
tools:text="This is a test description. Descriptions can be quite looooong." />

4
app/src/main/res/layout/item_account_field.xml
View file

@ -16,6 +16,7 @@
android:lineSpacingMultiplier="1.1"
android:textColor="?android:textColorPrimary"
android:textSize="?attr/status_text_medium"
android:textDirection="firstStrong"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
app:layout_constraintWidth_percent=".3"
@ -31,10 +32,11 @@
android:lineSpacingMultiplier="1.1"
android:textIsSelectable="true"
android:textSize="?attr/status_text_medium"
android:textDirection="firstStrong"
app:layout_constrainedWidth="true"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toEndOf="@+id/accountFieldName"
app:layout_constraintTop_toTopOf="parent"
tools:text="Field content. This can contain links and custom emojis" />
</androidx.constraintlayout.widget.ConstraintLayout>
</androidx.constraintlayout.widget.ConstraintLayout>