You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thank you for this beautiful tool!
I want to offer some improvement for compatibility heading links with GitHub's ones.
GitHub way:
Heading links anchor have no.toLowerCase() transform.
So, if you open this page, you see generated 'Минимальная настройка' heading url with hash, that have first letter in Upper case.
So, in any pages I can use this in this format quick.md#Минимальная настройка. For example, I use this in top of the same page, where I get something like ToC. This works on GitHub, but not works on generated Docsify's pages.
Docsify way
Heading links anchor have.toLowerCase() transform.
So, on this page you can see, that generated heading link anchor have first letter in lower case.
So, if you scroll to top of page, you see, that link does not works.
Try workaround
If change link in ToC to lewercased first letter, it works on Docsify, but not works on GitHub.
Thank you for this beautiful tool!
I want to offer some improvement for compatibility heading links with GitHub's ones.
GitHub way:
Heading links anchor have no
.toLowerCase()
transform.So, if you open this page, you see generated 'Минимальная настройка' heading url with hash, that have first letter in Upper case.
So, in any pages I can use this in this format
quick.md#Минимальная настройка
. For example, I use this in top of the same page, where I get something like ToC. This works on GitHub, but not works on generated Docsify's pages.Docsify way
Heading links anchor have
.toLowerCase()
transform.So, on this page you can see, that generated heading link anchor have first letter in lower case.
So, if you scroll to top of page, you see, that link does not works.
Try workaround
If change link in ToC to lewercased first letter, it works on Docsify, but not works on GitHub.
Possible solution
I think its enough just to delete this line: https://github.com/QingWei-Li/docsify/blob/master/src/core/render/slugify.js#L8
or maybe somewhere more,
or maybe you want to implement an option.
Or! just write some documentation how to do this.
The text was updated successfully, but these errors were encountered: