-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
fix: removing unnecessary empty paragraph check #2624
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
base: develop
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR refactors the blockquote compiler by removing unnecessary code and simplifying the return statement. The changes improve code cleanliness without altering functionality.
- Removed logic for filtering empty paragraphs from blockquote tokens
- Simplified the return statement by eliminating an intermediate variable
Comments suppressed due to low confidence (1)
src/core/render/compiler/blockquote.js:1
- The removal of the empty paragraph filtering logic should be verified with test coverage to ensure this doesn't introduce regressions in blockquote rendering, particularly for callout blocks with empty leading paragraphs.
export const blockquoteCompiler = ({ renderer }) =>
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thanks very much for taking a look at this @sy-records ! Unfortunately another minor issue has appeared with this fix, here is the updated Codebox: While the first line is now displayed, there seems to be an extra line now displayed before it. For example, with RC-3 notice the icon lines up with the first line of text:
And with this fix now the icon is above the first line of text:
Any ideas? Please let me know if more testing or info helps. |
|
I've retested with the updated PR and while the multi-line issues looks to be addressed a new issue has appeared where the space between bold and non-bold text is removed:
You can see this live at https://docsify-preview-61gtha3je-docsifyjs.vercel.app/preview/#/ui-kit?id=callouts I also retested with the original codebox and can confirm that the multline text now aligns with the callout icon as expected: Thanks very much, Paul. |



Summary
Related issue, if any:
Fix #2623
What kind of change does this PR introduce?
For any code change,
Does this PR introduce a breaking change?
Tested in the following browsers: