Skip to content
This repository was archived by the owner on Oct 4, 2023. It is now read-only.

[C-424] Fix profile screen bio links on android#1301

Merged
sliptype merged 3 commits into
mainfrom
sk-c-424-android-bio-links
May 10, 2022
Merged

[C-424] Fix profile screen bio links on android#1301
sliptype merged 3 commits into
mainfrom
sk-c-424-android-bio-links

Conversation

@sliptype

Copy link
Copy Markdown
Contributor

Description

  • Fixes the positioning and functionality of bio links on android
    Screenshot_20220510-141739
  • Fixes issue without text cut off due to numberOfLines=2

Dragons

This was already pretty fragile code and now it's even more fragile. Have to use measureInWindow on the refs of the links to get the position instead of onLayout because onLayout doesn't return correct values on android

How Has This Been Tested?

iOS and android

How will this change be monitored?

TestFlight, play store

@Kyle-Shanks Kyle-Shanks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@sliptype sliptype merged commit cf5fce4 into main May 10, 2022
@sliptype sliptype deleted the sk-c-424-android-bio-links branch May 10, 2022 20:11
@sliptype sliptype mentioned this pull request May 13, 2022
@AudiusProject AudiusProject deleted a comment from linear Bot Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants