Feature renderer #301
Feature renderer #301
Conversation
At markedjs#129, it only contains block level renderers. This patch contains both block level and span level renderers. Since renderer feature can do many things, I removed highlight option and headerPrefix option. markedjs#129
will be no id on headings.
Remove documentation on langPrefix. Add documentation on renderer.
|
I actually just merged everything by hand. It wasn't that bad. Just curious, why was the async highlighting removed? |
|
Closing. I added the async highlighting back because:
|
|
Anyway, I've made a few changes. Everything is working well. Great work. :) If you want to make anymore changes, I suggest pulling from upstream. Everything is there now. |
|
Ah, okay, I see why you removed highlight for sync highlighting, but users will have to be expected to do the async highlighting by hand now. We should probably open another issue debating this whole thing. I really like the approach of removing options that are obsolete now, but highlight is a pretty core option by now. A lot of people would be confused. |
|
@chjj Would you mind creating a new version tag with this feature in it? |
I started this at commit: 0d27d58
Related issue: #129