Skip to content

[FIX] Emoji detection at line breaks#13447

Merged
sampaiodiego merged 6 commits intoRocketChat:developfrom
savish28:Issue-#13059-Emoji-detection-linebreak
Feb 14, 2019
Merged

[FIX] Emoji detection at line breaks#13447
sampaiodiego merged 6 commits intoRocketChat:developfrom
savish28:Issue-#13059-Emoji-detection-linebreak

Conversation

@savish28
Copy link
Contributor

Closes #13059

Initially emojis at linebreaks were not detected.
rocketchat issue 13059 before

This PR solves the issue and now it looks like:
rocketchat issue 13059 after

Input given in both the cases:
:D foo :D bar :D baz :D quux :D
:D this :D is :D a simple :D test :D
without :D leading :D or :D trailing :D emote
:D hello :D world :D

(contains no trailing or leading whitespace)

@tassoevan tassoevan self-requested a review February 12, 2019 13:35
@tassoevan tassoevan assigned tassoevan and unassigned tassoevan Feb 12, 2019
@tassoevan tassoevan added type: bug area: ui Touches the code on client side labels Feb 12, 2019
@tassoevan tassoevan added this to the 0.75.0 milestone Feb 12, 2019
@sampaiodiego sampaiodiego changed the title [FIX] emoji detection at line breaks #13059 [FIX] Emoji detection at line breaks Feb 14, 2019
@sampaiodiego sampaiodiego merged commit 7f60daf into RocketChat:develop Feb 14, 2019
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Touches the code on client side type: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Emote detection fails around linebreaks

3 participants

Comments