Skip to content

fix(embed): strip front matter from included markdown#2752

Open
sy-records wants to merge 1 commit into
docsifyjs:developfrom
sy-records:fix/front-matter
Open

fix(embed): strip front matter from included markdown#2752
sy-records wants to merge 1 commit into
docsifyjs:developfrom
sy-records:fix/front-matter

Conversation

@sy-records

Copy link
Copy Markdown
Member

Summary

Related issue, if any:

Close #2654

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

For any code change,

  • Related documentation has been updated, if needed
  • Related tests have been added or updated, if needed

Does this PR introduce a breaking change?

  • Yes
  • No

Tested in the following browsers:

  • Chrome
  • Firefox
  • Safari
  • Edge

@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

@sy-records is attempting to deploy a commit to the Docsify Team on Vercel.

A member of the Team first needs to authorize it.

@sy-records sy-records requested a review from paulhibbitts July 7, 2026 07:31
@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docsify-preview Ready Ready Preview, Comment Jul 7, 2026 7:32am

@paulhibbitts paulhibbitts left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks very much @sy-records ! I was unsure how to duplicate this issue on Codesandbox, so I used the provided PR preview and screenshots from the originating issue.

I reviewed the original PR and confirmed the issue is on the most recent RC.4 release:
https://preview.docsifyjs.org/#/embed-files

Looking at the PR preview, I can confirm that the included Markdown is now stripped and is not included in the rendered Sidebar:
https://docsify-preview-git-fork-sy-records-fix-front-matter-docsifyjs.vercel.app/preview/#/embed-files

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.

Change md include naming to improve menu headers

2 participants