Skip to content
New issue

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

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

Already on GitHub? # to your account

MarkdownColumn should accept markdown configuration #184

Closed
jace opened this issue Apr 20, 2018 · 2 comments
Closed

MarkdownColumn should accept markdown configuration #184

jace opened this issue Apr 20, 2018 · 2 comments

Comments

@jace
Copy link
Member

jace commented Apr 20, 2018

A markdown column may specifically need to allow sanitized HTML, or be pure Markdown, or something else of a variety of configuration options allowed by the Markdown processor.

  1. Class MarkdownComposite accepts an options={} parameter as the last item
  2. Class MarkdownColumn also accepts options={} (position irrelevant, but last for consistency)
  3. MarkdownColumn calls composite with functools.partial(MarkdownComposite, options=options)
  4. MarkdownComposite.__setattr__ calls markdown(value, **self.options) (pending change in Markdown processor in Replace GFM module with JS version #182)
@jace
Copy link
Member Author

jace commented Sep 18, 2019

Related: Google has augmented rel="nofollow" with a new value ugc to indicate user generated content. This should be configurable by column or possibly by form.

@jace
Copy link
Member Author

jace commented Aug 2, 2020

Turns out composite won't accept partial(MarkdownComposite) as a composite class.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

1 participant