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

Format markdown component content #63

Merged
merged 2 commits into from
Nov 24, 2021

Conversation

antonyfaris
Copy link
Member

Changes

Fixes the format of markdown inside a markdown component.
Fixes #41
Fixes #42

Testing

I added an extra test.

Docs

@matthewp
Copy link

Looks good!

Copy link
Collaborator

@jasikpark jasikpark left a comment

Choose a reason for hiding this comment

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

Looks good to me!

if (isDocCommand(formattedStyles)) return formattedStyles;
formattedStyles = formattedStyles[0];

formattedStyles = stripTrailingHardline(formattedStyles);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh! I appreciate the function instead of the other logic

@jasikpark
Copy link
Collaborator

Thanks for your patience with my review of it - sorry for letting it sit for a bit

@jasikpark jasikpark merged commit f7cf7c1 into withastro:main Nov 24, 2021
@antonyfaris antonyfaris deleted the fix/markdown-content branch November 24, 2021 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants