Closed langleyd closed 2 months ago
Attention: Patch coverage is 76.75676%
with 43 lines
in your changes missing coverage. Please review.
Project coverage is 90.14%. Comparing base (
4c50704
) to head (0bb2c9e
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
2 New issues
0 Accepted issues
Measures
0 Security Hotspots
98.8% Coverage on New Code
0.0% Duplication on New Code
What's included this PR?
Custom Suggestion Pattern API and Auto-replace plain text with emoji feature(Web only so far). E.g.
:) => 🙂
What does it look like?
https://github.com/matrix-org/matrix-rich-text-editor/assets/6135282/0a7e6ccf-d303-464f-bc79-6c01d921dd90
Implementation detail
I considered 2 approaches for this:
/
,@
,#
). The advantage of this is that it builds upon an existing pattern rather than creating a new way of doing things. It also gives flexibility to the client as to how the feature works. The version implemented works as the web-client does, by automatically replacing the suggestions, but there has been a suggestion previously that it should work as it does on Telegram where suggestions are rendered first from which the user can choose to accept: