Skip to content
This repository was archived by the owner on Jun 13, 2024. It is now read-only.

Add custom list rendering for body frontmatter#48

Open
ELD wants to merge 2 commits into
mainfrom
custom-renderer-collection-body
Open

Add custom list rendering for body frontmatter#48
ELD wants to merge 2 commits into
mainfrom
custom-renderer-collection-body

Conversation

@ELD

@ELD ELD commented Sep 6, 2023

Copy link
Copy Markdown
  • Set the list style type based on whether the list is ordered or unordered
  • Set list-style-position to inside to avoid weird layouts when text is centered

This makes #42 render correctly and is broken out from that PR as discussed there.

@netlify

netlify Bot commented Sep 6, 2023

Copy link
Copy Markdown

Deploy Preview for configtips ready!

Name Link
🔨 Latest commit 3403002
🔍 Latest deploy log https://app.netlify.com/sites/configtips/deploys/64fb55cc6a047f00086079c7
😎 Deploy Preview https://deploy-preview-48--configtips.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ELD

ELD commented Sep 6, 2023

Copy link
Copy Markdown
Author

It looks like CI failures are due to files being malformed upstream?

@ELD
ELD marked this pull request as ready for review September 6, 2023 16:57
@ELD
ELD requested a review from KyleTryon as a code owner September 6, 2023 16:57
@KyleTryon

Copy link
Copy Markdown
Contributor

It looks like CI failures are due to files being malformed upstream?

Im not sure how those made it in.. just fixed it though. rebase away!

@ELD ELD mentioned this pull request Sep 6, 2023
Eric Dattore added 2 commits September 8, 2023 11:03
- Set the list style type based on whether the list is ordered or
  unordered
- Set list-style-position to inside to avoid weird layouts when text is
  centered
@ELD
ELD force-pushed the custom-renderer-collection-body branch from 085845e to 3403002 Compare September 8, 2023 17:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants