| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
@jacobsorme is attempting to deploy a commit to the Docsify Team on Vercel. A member of the Team first needs to authorize it. |
Sorry, something went wrong.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sorry, something went wrong.
There was a problem hiding this comment.
This PR adds a new :fragmentFullLine option that enables full-line exclusion of fragment identifier lines when embedding file fragments. When enabled, the entire line containing the fragment marker (e.g., /// [demo] comment text) is excluded from the embedded content, not just the marker itself.
Key Changes:
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file| File | Description |
|---|---|
| test/integration/example.test.js | Added integration test verifying full-line fragment exclusion behavior |
| src/core/render/embed.js | Modified extractFragmentContent to accept and handle fullLine parameter with updated regex logic |
| src/core/render/compiler.js | Added fragmentFullLine property assignment to embed configuration |
| src/core/render/compiler/media.js | Fixed grammar in video and audio error messages |
| docs/embed-files.md | Documented the new :fragmentFullLine option |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sorry, something went wrong.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
LGTM.
A tiny suggest is we may name the config as omitFragmentLine (as well as within the codebase) which seems more understandable for the purpose.
Sorry, something went wrong.
Co-authored-by: Koy Zhuang <koyzhuang@qq.com>
There was a problem hiding this comment.
LGTM, thx!
Sorry, something went wrong.
|
Can you all please add an example to the docs? It isn't clear from the update doc what exactly this does. The explanation is too simple. |
Sorry, something went wrong.
|
Sure thing, will look into example + better formulated explanation. This is the look ATM
The use case for me is specifically this - I want to both use a HTML file for a Docsify snippet, and be able to render the file stand-alone. Neither<!-- /// [demo] --> nor /// [demo] solves both these cases, the first gives me a --> in Docsify and the latter a /// [demo] in the stand-alone page.
How about something like that + a short HTML example & Docsify markdown embed line? |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
New option :fragmentFullLine :omitFragmentLine that when used will enable full-line ignore of file embed fragment lines.
Related issue, if any:
What kind of change does this PR introduce?
For any code change,
Does this PR introduce a breaking change?
Tested in the following browsers: