feat(ios): support for html in AttributedStrings

[Brianggalvez]

Added support for html key when creating an AttributedString. If html is not present it fallbacks to previous behavior

[m1ga]

Hi, thanks for the contribution. Can you add a simple example please so people can quickly test it.

And have a look at https://github.com/tidev/titanium_mobile/blob/master/apidoc/Titanium/UI/AttributedString.yml and ad your property so it is visible in the documentation.

Check this page https://github.com/tidev/organization-docs/blob/main/AUTHORIZED_CONTRIBUTORS.md on how to become a contributor.

[hansemannn]

Once the CLA is signed, we can go ahead!

[hansemannn]

The code changes look fine

[Brianggalvez]

Once the CLA is signed, we can go ahead!

I've signed on Wednesday, don't know if I need to do anything else

[hansemannn]

cc @joshualambert ?

[joshualambert]

Confirmed -- we now have a CLA signature for @Brianggalvez. Added him to authorized CLA list here: https://github.com/tidev/organization-docs/blob/main/AUTHORIZED_CONTRIBUTORS.md

CC: @Brianggalvez @hansemannn @m1ga

[hansemannn]

The CLA action seems to have an issue:

Run tidev/tidev-cla-action@v1
User Brianggalvez for commit 3ab3a60b1a50416d38083034f135990130d2499d is authorized
User Brianggalvez for commit 02b21cabc076df800f86adc18fed236e2623b31c is authorized
file:///home/runner/work/_actions/tidev/tidev-cla-action/v1/index.js:37
for (const { author: { login }, sha } of pr.data) {
                       ^

TypeError: Cannot read properties of null (reading 'login')
    at file:///home/runner/work/_actions/tidev/tidev-cla-action/v1/index.js:37:24
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Maybe because of Brianggalvez vs brianggalvez? Mhh.

[hansemannn]

I've tested a bit with this PR (it works great!) and noticed that on Android, there is the "html" property on the label itself already. I am wondering if iOS could potentially also use this property and construct the NSAttributedString internally. That way, it's 100 % cross platform! Aside that, other attributes in NSAttributedString will be ignored anyways, so we could have one single API for both.

[hansemannn]

@Brianggalvez I've added the suggested changes in this PR to your branch: Brianggalvez#38

Great new API!