[Story] Add/edit/delete a link using the RTE in EX

[jonnyandrew]

Description

As a user

• I want to add a link to text in the rich text editor
• so that I can send messages containing links

As a user

• I want to edit or delete a link within the rich text editor
• so that I can change or fix links in my own messages

Design

• TBC

Size estimate

M

Dependencies

• [Story] RTE Rich text mode on EX #1937

Acceptance criteria

• User can add a link to selected text in the rich text editor.
• User can edit a link in the rich text editor.
• User can delete a link in the rich text editor.

Out of scope

• Edit link - allow editing both URL and displayed text - iOS matrix-org/matrix-rich-text-editor#616
• Edit link - allow editing both URL and displayed text - Android matrix-org/matrix-rich-text-editor#617
• Edit link - allow editing both URL and displayed text - Web matrix-org/matrix-rich-text-editor#618

Nothing

Android

• Design sign-off on completion
• QA sign-off on completion
• Product sign-off on completion

iOS

• Design sign-off on completion
• QA sign-off on completion
• Product sign-off on completion

Subtasks

Android

Preview Give feedback

1. [Rich text editor] Add link functionality to rich text editor vector-im/element-x-android#1309
   +0

iOS

Preview Give feedback

No tasks being tracked yet.

Library

Preview Give feedback

1. [Android] Add link support to compose library matrix-org/matrix-rich-text-editor#801
   Android +1

[callumu]

When editing a link on Android, only the link can be edited, not the word that's had the link added to it. This is different to iOS. @jonnyandrew I feel like we discussed this as an Android limitation, but can you confirm?

[jonnyandrew]

Yes this is a known limitation but my understanding was that it affected all platforms as it is not yet implemented in the rust model (see open issues matrix-org/matrix-rich-text-editor#616, matrix-org/matrix-rich-text-editor#617, matrix-org/matrix-rich-text-editor#618). Are you sure it's working correctly on iOS @callumu? If so, perhaps iOS is using a workaround to achieve the behaviour @alfogrillo?

[callumu]

Yes this is a known limitation but my understanding was that it affected all platforms as it is not yet implemented in the rust model (see open issues matrix-org/matrix-rich-text-editor#616, matrix-org/matrix-rich-text-editor#617, matrix-org/matrix-rich-text-editor#618). Are you sure it's working correctly on iOS @callumu? If so, perhaps iOS is using a workaround to achieve the behaviour @alfogrillo?

Oh sorry this is my fault, it doesn't exist on iOS also. Ignore my initial comment and I'll mark this as done

[jonnyandrew]

No problem! Thanks for reviewing.