Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Implement starter link support #437

Merged
merged 3 commits into from
Sep 5, 2016
Merged

Implement starter link support #437

merged 3 commits into from
Sep 5, 2016

Conversation

kegsay
Copy link
Member

@kegsay kegsay commented Sep 2, 2016

This involves modal dialogs because browsers are terrible. Also did a bit of refactoring, moving lots of Scalar specific stuff out of RoomSettings.

var body = HtmlUtils.bodyToHtml(content, this.props.highlights, {});

if (this.props.highlightLink) {
body = <a href={ this.props.highlightLink }>{ body }</a>;
}
else if (content.data && typeof content.data["org.matrix.neb.starter_link"] === "string") {
// FIXME: React replaces this with a <span> because there is no href= - Matthew haaaalp!
Copy link
Member

Choose a reason for hiding this comment

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

i fix this by adding an href="#" and cancelling the event in the onClick.

@ara4n
Copy link
Member

ara4n commented Sep 4, 2016

lgtm other than comment.

@kegsay kegsay merged commit 6e2fd85 into develop Sep 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants