Skip to content

[FIX] Marked parser breaking announcements and mentions at the start of messages#11357

Merged
ggazzo merged 2 commits intoRocketChat:developfrom
vynmera:fix-announcement
Jul 21, 2018
Merged

[FIX] Marked parser breaking announcements and mentions at the start of messages#11357
ggazzo merged 2 commits intoRocketChat:developfrom
vynmera:fix-announcement

Conversation

@vynmera
Copy link
Copy Markdown
Contributor

@vynmera vynmera commented Jul 5, 2018

Closes #11352
Closes #11310
Closes #10949
Closes #11392

2018-07-05_16-26-57
2018-07-05_16-31-30

@ggazzo ggazzo added the area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction label Jul 6, 2018
@ggazzo ggazzo added this to the 0.67.0 milestone Jul 6, 2018
@nickbroon
Copy link
Copy Markdown

It would be great to see this in both a 0.66.x fix release and 0.67

@theorenck theorenck modified the milestones: 0.67.0, 0.68.0 Jul 19, 2018
@ggazzo ggazzo requested a review from tassoevan July 20, 2018 16:40
if (Meteor.isClient) {
Blaze.registerHelper('RocketChatMarkdown', text => Markdown.parse(text));
Blaze.registerHelper('RocketChatMarkdownUnescape', text => Markdown.parseNotEscaped(text));
Blaze.registerHelper('RocketChatMarkdownWithoutP', (text) => {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think RocketChatMarkdownInline communicate better the result we're getting from this helper.

@vynmera
Copy link
Copy Markdown
Contributor Author

vynmera commented Jul 20, 2018

@tassoevan 👍

Copy link
Copy Markdown
Member

@tassoevan tassoevan left a comment

Choose a reason for hiding this comment

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

Very nice!

@ggazzo ggazzo merged commit 92f60c0 into RocketChat:develop Jul 21, 2018
@sampaiodiego sampaiodiego mentioned this pull request Jul 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants