@hikerpig thanks for the cool PR, can you add a setting to disable this feature? Some people may not need auto generated toc

@ZeroX-DG I don't see your worry here, maybe I misunderstand it.

Let's assume if someone typed this [TOC] placeholder - which is not a common word - manually, the expected behavior should be auto generating content.

Say if there is a config and it is turned off, what is the expected behavior for the placholder?

@hikerpig You can totally ignore my comment, I didn't see the [toc] tag. My brain sometimes is on autopilot 😄

Code review could be exhausting sometimes 😆 , thank you for your work.

Let me know if there is anything else that needs to be updated.

looks nice.

However we should remove current implementation of TOC feature before add this new one.

@amedora can you share your thought on this? Why can't they both exist?

@hikerpig it seem that your markdownItTocAndAnchor lib doesn't support heading with uppercase letter. Can you fix it please?

@ZeroX-DG ,Technically both can exist together.
However, it's questionable to have two implementations for that almost same feature...(is it worth to maintain both codes?)

@amedora I think we should also keep the old TOC too, because user will have more control/customization of the TOC with the old feature.

@ZeroX-DG, I see. Thank you for your clarification 👍

@hikerpig ping!

@Flexo013 What needs to be changed? Should I add anything ?

There was this comment by Zero:

@hikerpig it seem that your markdownItTocAndAnchor lib doesn't support heading with uppercase letter. Can you fix it please?

@ZeroX-DG I can't reproduce the bug what you described with heading with uppercase letter.

Can you be more specific? Give me some example content so I can improve against. 😃

@hikerpig Here is a small example:

As you can see, the id of the actual heading has uppercase letter while in the link it automatically lowercase everything, this make the link not navigate to the heading.

@ZeroX-DG Oh I see, I will fix this in 48h.

@ZeroX-DG I've updated the package to fix this annoying lowercase-casting thing.

@hikerpig I want to merge this very soon. Could you fix the conflicts?

@hikerpig Could you fix CI error too? I think updating snapshot should be enough! Check https://travis-ci.org/BoostIO/Boostnote/builds/596037299#L394

Error fixed.

BTW, I suggest that the npm 'test' script should contain jest tests.

@hikerpig Thanks! ❤️