Skip to content

Stop react-markdown from converting "[foo]" to a link.#151

Merged
jrhone merged 6 commits intostreamlit:developfrom
jrhone:markdown_links
Sep 24, 2019
Merged

Stop react-markdown from converting "[foo]" to a link.#151
jrhone merged 6 commits intostreamlit:developfrom
jrhone:markdown_links

Conversation

@jrhone
Copy link
Copy Markdown
Contributor

@jrhone jrhone commented Sep 20, 2019

Issue:

#150

Decription:

Stop react-markdown from converting "[foo]" to a link.

  • added a renderer to ReactMarkdown for link-references that detects a missing url and doesn't convert to a link.

  • By checking this box, I agree that all contributions to this project are made under the Apache 2.0 license.

@jrhone jrhone requested a review from a team September 20, 2019 13:50
@tvst tvst changed the title Markdown links Stop react-markdown from converting "[foo]" to a link. Sep 23, 2019
)

const linkReferenceHasParens = (props: LinkReferenceProps): any => {
console.log(props)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove console.log. If you really want to log this for future debugging, use logMessage from lib/log

</a>
)

const linkReferenceHasParens = (props: LinkReferenceProps): any => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add comment explaining what this function does. It's not obvious.

@jrhone jrhone merged commit 1c35e83 into streamlit:develop Sep 24, 2019
tconkling added a commit to tconkling/streamlit that referenced this pull request Sep 25, 2019
# By Jonathan Rhone (3) and others
# Via GitHub
* develop:
  Text changes in ☰ menu and several CLI prompts (streamlit#180)
  add format option for slider widget and return texts for display values (streamlit#154)
  Merge ForwardMsg caching into develop  (streamlit#178)
  e2e test for magic (streamlit#172)
  Bart example update (streamlit#179)
  Pedantic CircleCI cleanup (streamlit#174)
  Fix overwriting elements in the sidebar (streamlit#181)
  update vega lite chart snapshot (streamlit#183)
  Stop react-markdown from converting "[foo]" to a link. (streamlit#151)
  Issue 1204: Graphs using Vega-Lite (streamlit#56)
  Removing uber demo from streamlit repo (streamlit#159)

# Conflicts:
#	lib/streamlit/DeltaGenerator.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants