Skip to content

[FIX] Fix rendering of links in the announcement modal#13250

Merged
engelgabriel merged 4 commits intoRocketChat:developfrom
supra08:announcement-modal-link
Apr 6, 2019
Merged

[FIX] Fix rendering of links in the announcement modal#13250
engelgabriel merged 4 commits intoRocketChat:developfrom
supra08:announcement-modal-link

Conversation

@supra08
Copy link
Copy Markdown
Contributor

@supra08 supra08 commented Jan 24, 2019

Closes #8021

@tassoevan @ggazzo In this PR, the links in the announce modal have been made clickable. I decided to use regex to make it identfy and render.

screenshot from 2019-01-24 20-06-12

@tassoevan tassoevan requested review from ggazzo and tassoevan January 24, 2019 17:28
@tassoevan tassoevan added area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction type: improvement labels Jan 24, 2019
@ggazzo ggazzo added this to the 0.75.0 milestone Feb 5, 2019
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.

Suggestion: use the same autolinker that parses links in messages.

@rodrigok
Copy link
Copy Markdown
Member

rodrigok commented Mar 4, 2019

A better approach would be allowing markdown via our MD engine.

@engelgabriel engelgabriel requested a review from tassoevan March 22, 2019 11:51
@engelgabriel
Copy link
Copy Markdown
Member

@rodrigok can we merge this?

@tassoevan tassoevan dismissed their stale review March 25, 2019 20:57

I did some changes myself

@engelgabriel engelgabriel merged commit c8b0262 into RocketChat:develop Apr 6, 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/ux Related to UI/UX, frontend code, accessibility, and user interaction type: improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants