Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Render description field in Markdown #39

Merged
merged 1 commit into from
Dec 23, 2019
Merged

Conversation

AntoineAugusti
Copy link
Member

@AntoineAugusti AntoineAugusti commented Dec 23, 2019

Fixes #32

image

@AntoineAugusti AntoineAugusti changed the title Render field description in Markdown Render description field in Markdown Dec 23, 2019
Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

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

Thoughts for later: we probably should factorize more things between each field component. You just made 2 PRs that made the same set of changes in 3 different files.

@AntoineAugusti
Copy link
Member Author

True. There is room for improvements.

I'm sure the form group thing can go in a shared component and we can leverage templates. I'll open a follow-up refactor issue 👌

@AntoineAugusti AntoineAugusti merged commit 03188f2 into master Dec 23, 2019
@AntoineAugusti AntoineAugusti deleted the markdown-description branch December 23, 2019 18:31
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.

Support Markdown in field descriptions
2 participants