Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added markdown parsing to textarea labels, in the spirit of f19a65b7d… #281

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

albapa
Copy link

@albapa albapa commented Oct 4, 2018

This is a proposed solution to the problem in

https://discourse.getgrav.org/t/markdown-in-form-label-to-add-page-link/5386/8

i.e. markdown parsing in textarea labels. It's basically copy-pasta of the relevant parts in f19a65b

<div class="{{ form_field_wrapper_classes ?: 'form-textarea-wrapper' }} {{ field.size }} {{ field.wrapper_classes }}">
<label style="display:inline;" class="inline" for="{{ id|e }}">
Copy link
Member

Choose a reason for hiding this comment

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

What is the purpose of putting the label in this input block and not the label block? This will change the display on all forms with textarea as the div/class structure is changed.

Copy link
Author

Choose a reason for hiding this comment

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

Yes, that's right. I've rearranged the code now. Let me know if that looks better!

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