Skip to content

Support multiple markdown processor(redcarpet, pandoc)#15

Open
zonuexe wants to merge 10 commits into
niku:masterfrom
zonuexe:feature/multi-engine
Open

Support multiple markdown processor(redcarpet, pandoc)#15
zonuexe wants to merge 10 commits into
niku:masterfrom
zonuexe:feature/multi-engine

Conversation

@zonuexe
Copy link
Copy Markdown
Contributor

@zonuexe zonuexe commented Jan 17, 2016

No description provided.

- markdown-preview-eww-major-mode-default-dialect
- markdown-preview-eww-major-mode-dialect-alist
@zonuexe zonuexe changed the title [WIP]Feature/multi engine [WIP]Support multiple markdown processor(redcarpet, pandoc) Jan 17, 2016
@niku
Copy link
Copy Markdown
Owner

niku commented Jan 18, 2016

I'm looking forward it! :)

@tzz
Copy link
Copy Markdown

tzz commented Feb 5, 2016

This would resolve #12

Comment thread markdown-preview-eww.el Outdated
(format "require \"redcarpet\"
(defcustom markdown-preview-eww-convert-command
nil
"Symbol of markdown convert function.")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If this is a function name, the variable name and defcustom should be as follows (note the default). You should defgroup a customization group and put all defcustoms under it to make the user experience better.

Even nicer to the user is if you give them a choice between the known conversion functions and an arbitrary function.

(defcustom  markdown-preview-eww-convert-function 'markdown-preview-eww-convert-command-redcarpet
  "*A function that is called to convert the Markdown document to HTML that eww can render"
  :group 'markdown-preview-eww
  :type 'function)

@tzz
Copy link
Copy Markdown

tzz commented Feb 5, 2016

What's needed to merge this pull request?

@tzz tzz mentioned this pull request Feb 5, 2016
@zonuexe zonuexe force-pushed the feature/multi-engine branch from 92f5a01 to 241dcf4 Compare February 14, 2016 15:34
@zonuexe zonuexe force-pushed the feature/multi-engine branch from 2f8f7b1 to 224b68f Compare February 27, 2016 15:52
@zonuexe
Copy link
Copy Markdown
Contributor Author

zonuexe commented Feb 27, 2016

@niku @tzz
Hi, I have implemented this PR.
I wanted to save the position on the screen, but does not work well. Other function seems to work.

@zonuexe zonuexe changed the title [WIP]Support multiple markdown processor(redcarpet, pandoc) Support multiple markdown processor(redcarpet, pandoc) Feb 27, 2016
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.

3 participants