Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

abbreviations support #5

Open
blaenk opened this issue Jun 16, 2015 · 2 comments
Open

abbreviations support #5

blaenk opened this issue Jun 16, 2015 · 2 comments

Comments

@blaenk
Copy link

blaenk commented Jun 16, 2015

I'm not sure how common this one is, but I first used it in kramdown, Pandoc has support for parsing them but not yet (last I checked) for emitting them, so I used to write my own Pandoc AST transformer to emit them manually, and now that I'm not using Pandoc I do a regex search and replace. I imagine there might be advantages to having this be built into something like pulldown-cmark, in terms of speed, perhaps it enables doing it in the same pass that pulldown-cmark is doing, instead of a separate pass.

@blaenk blaenk changed the title abbreviations abbreviations support Jun 16, 2015
@marcusklaas
Copy link
Collaborator

marcusklaas commented May 25, 2021

This seems like a sensible feature, that shouldn't be too complex to add. There is however the issue of having to do essentially full text search for all the defined abbreviations. If it's hidden behind an option it's probably acceptable, but we should be very explicit to users about the impact on performance.

@ulab
Copy link

ulab commented Aug 29, 2024

Just a little heads up: Usually you only need to find the first time an abbreviation is used on a page.

Perhaps the option can be "off, first, all"?

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

No branches or pull requests

4 participants