Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Allow HTML tags to be used for styling in bigText field only #2067

Merged
merged 2 commits into from
Aug 30, 2021

Conversation

silencer07
Copy link
Contributor

@silencer07 silencer07 commented Jun 26, 2021

Rationale:

bigText field should be allowed to have HTML tags as a way to format the message since this has been already supported in android natively for a long time

Expected behaviour:

Allows a subset of html tags to be used as outlined in this android documentation. Note that if user did not provided a bigText field but added html tags in message it self, it won't work as message field in iOS does not support HTML tags nor iOS has bigText property

Testing:

on bigText field, add some <br/> tags and see if it will put the subsequent string to the next line in android.

Change:

This is a small change, this merely decorates the bigText as Spanned Text

@silencer07
Copy link
Contributor Author

This PR is a modified and simplifed version of wanderbon's PR that is not yet merged

@Dallas62
Copy link
Collaborator

Hi,
Thanks for the PR, will not be available this month, will look when I comme back.
Regards

@silencer07
Copy link
Contributor Author

@Dallas62 hi can we merge this?

@silencer07
Copy link
Contributor Author

@Dallas62 hello?

@Dallas62 Dallas62 changed the base branch from master to dev August 30, 2021 21:27
@Dallas62 Dallas62 merged commit 57be563 into zo0r:dev Aug 30, 2021
@Dallas62
Copy link
Collaborator

Hi @silencer07
Thanks for your contribution !
Regards

@silencer07
Copy link
Contributor Author

Thanks @Dallas62

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants