Skip to content

fix(android, ios): Link copy-pasting inconsistencies#640

Open
hejsztynx wants to merge 4 commits into
mainfrom
@ksienkiewicz/fix-link-copy-pasting
Open

fix(android, ios): Link copy-pasting inconsistencies#640
hejsztynx wants to merge 4 commits into
mainfrom
@ksienkiewicz/fix-link-copy-pasting

Conversation

@hejsztynx

Copy link
Copy Markdown
Collaborator

Summary

There were some inconsistencies when copy-pasting created links

iOS:

  • when you add a link via setting input's value or copy-pasting an already existing manual link where href == textValue, but the href does not match the given linkRegex, it gets marked as an autolink, which immediately removes the link, as the regex match fails

Android:

  • copying pasting an autolink resulted in creating a manual link unnecessarily

Videos

The said iOS bug

Screen.Recording.2026-06-15.at.18.34.30.mov

Compatibility

OS Implemented
iOS
Android

Checklist

  • E2E tests are passing
  • Required E2E tests have been added (if applicable)

Copilot AI review requested due to automatic review settings June 15, 2026 16:37
@hejsztynx hejsztynx force-pushed the @ksienkiewicz/fix-link-copy-pasting branch from 2accb4b to 056004f Compare June 15, 2026 16:38

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI 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.

Pull request overview

Copilot reviewed 12 out of 17 changed files in this pull request and generated 1 comment.

Comment thread ios/styles/LinkStyle.mm Outdated
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