Skip to content

Conversation

@Mansi449
Copy link
Contributor

Description

Fixes #1258
Used CDATA to use HTML tags.

Tests performed

Tested on Motorola Moto G(5) (Android 7.0 API 24)

@neslihanturan neslihanturan requested a review from misaochan March 27, 2018 12:49
translate.setText(Html.fromHtml(getString(R.string.about_translate)));
credits.setText(Html.fromHtml(getString(R.string.about_credits)));
faq.setText(Html.fromHtml(getString(R.string.about_faq)));

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CDATA changes look fine, but why the switch to TextViews?

Copy link
Contributor Author

@Mansi449 Mansi449 Mar 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the text is not set like this then it does not work as expected i.e. if we specify
<TextView android:text="@string/about_rate_us" />

then the output is:
https://drive.google.com/open?id=1YHZcHHhgmUBqMUE48BpWY5toeEcMgjqC

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, okay, looks good in that case. :)

@misaochan
Copy link
Member

Tested, looks good to me. Thanks @Mansi449 !

@misaochan misaochan merged commit ad0ba3e into commons-app:master Mar 28, 2018
@Mansi449 Mansi449 deleted the UsingHTML branch March 29, 2018 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants