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

CodeBlock - fix line numbers always shown when highlightLines is set #2474

Merged
merged 2 commits into from
Oct 7, 2024

Conversation

alex-ju
Copy link
Member

@alex-ju alex-ju commented Oct 2, 2024

๐Ÿ“Œ Summary

Uncouple hasLineNumbers from highlightLines.

๐Ÿ› ๏ธ Detailed description

With this change, if @hasLineNumbers={{false}} we highlight the lines without automatically showing the numbers.

Preview in showcase

๐Ÿ”— External links

Jira ticket: HDS-3348


๐Ÿ‘€ Component checklist

๐Ÿ’ฌ Please consider using conventional comments when reviewing this PR.

Copy link

vercel bot commented Oct 2, 2024

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Updated (UTC)
hds-showcase โœ… Ready (Inspect) Visit Preview Oct 7, 2024 6:30pm
hds-website โœ… Ready (Inspect) Visit Preview Oct 7, 2024 6:30pm

Copy link
Contributor

@KristinLBradley KristinLBradley left a comment

Choose a reason for hiding this comment

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

Added a comment with suggested improvement to changelog.

@KristinLBradley KristinLBradley requested review from a team October 2, 2024 17:58
@alex-ju alex-ju force-pushed the alex-ju/fix-code-block-highlight-lines branch from 5c70d8e to 0f3df00 Compare October 7, 2024 18:26
@alex-ju alex-ju merged commit b72869e into main Oct 7, 2024
14 checks passed
@alex-ju alex-ju deleted the alex-ju/fix-code-block-highlight-lines branch October 7, 2024 18:37
@hashibot-hds hashibot-hds mentioned this pull request Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants