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

feat: Line Numbers only for single code block #1363

Closed
wants to merge 8 commits into from

Conversation

g4rry420
Copy link
Contributor

@g4rry420 g4rry420 commented Sep 18, 2022

Signed-off-by: GurkiranSingh gurkiransinghk@gmail.com

feat: Line Numbers only for single code block

Please, let me know if you see any issues :)

g4rry420 and others added 3 commits September 18, 2022 16:42
Signed-off-by: GurkiranSingh <gurkiransinghk@gmail.com>
Signed-off-by: GurkiranSingh <gurkiransinghk@gmail.com>
const fence = md.renderer.rules.fence!
md.renderer.rules.fence = (...args) => {
const rawCode = fence(...args)
const [tokens, idx] = args
const token = tokens[idx]
const attr = token.attrs && token.attrs[0]
Copy link
Member

Choose a reason for hiding this comment

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

This logic won't work if markdown-it-attrs is disabled. As a fallback, you need to manually check token.info too for that option (refer highlightLines plugin for example, we had done similar stuff there). Also, have you checked if this works with line highlighting and snippets import? If yes, then consider adding some example to the docs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@brc-dd On disabling markdown-it-attrs, I noticed that on the top right corner of code block where we are displaying the language for the code, the {1,4,6-8} - line highlightning syntax is also coming up. Is this intended or a bug ?
Screenshot 2022-09-24 132412

Copy link
Member

Choose a reason for hiding this comment

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

Yeah it's a bug.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have resolved the above bug and also line numbers are now working even if markdown-it-attrs is disabled. Please, let me know if you see any issues :)

…ne numbers are compatible even when markdown-it-attrs is disabled. Also, added example of line numbers code block with line highlightning

Signed-off-by: GurkiranSingh <gurkiransinghk@gmail.com>
@brc-dd brc-dd closed this in #1733 Dec 24, 2022
@g4rry420 g4rry420 deleted the issue_1256 branch December 28, 2022 20:09
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Line Numbers Only for some Code Blocks
2 participants