Skip to content

Conversation

deep93333
Copy link
Contributor

@deep93333 deep93333 commented Oct 8, 2024

Summary

  • Update the dependencies to latest release (next-mdx-remote, mdx.js, rehype and remark plugins)
  • Add support for code line highlight and also made changes to examples accordingly.
  • Applied prettier using @mintlify/prettier-config

Example

Screenshot 2024-10-08 at 11 30 39 PM Screenshot 2024-10-08 at 11 25 59 PM

Test plan

To locally test this package in mint we need to link this project. To link this project i was using https://github.com/wclr/yalc i was struggling to link with yarn or npm. I found out mint project uses yarn 3 while mdx repo is using yarn 1 so there was compatibility issue.

  1. Install yalc globally
yarn global add yalc
  1. Publish it from mdx project
yalc publish
  1. Link it to mint project
yalc link @mintlify/mdx

Copy link

vercel bot commented Oct 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mdx-app-router ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2024 6:52pm
mdx-pages-router ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2024 6:52pm

@deep93333 deep93333 changed the title [DRAFT] Deep/eng 3060 code blocks line highlighting Deep/eng 3060 code blocks line highlighting Oct 9, 2024
@deep93333 deep93333 marked this pull request as ready for review October 9, 2024 03:44
@dks333
Copy link
Contributor

dks333 commented Oct 21, 2024

Quick question (might not be included in the initial design): Do we need to highlight codes without languages being specified?

Copy link
Contributor

@dks333 dks333 left a comment

Choose a reason for hiding this comment

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

Codes should be highlighted even if the language is not specified.

@dks333 dks333 merged commit db78d1b into main Oct 22, 2024
1 of 3 checks passed
@dks333 dks333 deleted the deep/eng-3060-code-blocks-line-highlighting branch October 22, 2024 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants