Skip to content
This repository was archived by the owner on Nov 2, 2023. It is now read-only.

Improve implementation draft list layout #533

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

davishmcclurg
Copy link
Contributor

@davishmcclurg davishmcclurg commented Aug 20, 2023

This adds a comma between draft and date-draft items and checks for empty arrays to prevent stray "draft-0" output.

Before:

Screen Shot 2023-08-21 at 10 47 42 AM

After:

Screen Shot 2023-08-21 at 10 47 44 AM

@netlify
Copy link

netlify bot commented Aug 20, 2023

Deploy Preview for condescending-hopper-c3ed30 ready!

Name Link
🔨 Latest commit 92bfe8c
🔍 Latest deploy log https://app.netlify.com/sites/condescending-hopper-c3ed30/deploys/64e3a3e7d6d14c0007c57eef
😎 Deploy Preview https://deploy-preview-533--condescending-hopper-c3ed30.netlify.app/implementations
📱 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.

@jdesrosiers
Copy link
Member

It's on purpose that 2019-09 and 2020-12 don't say "draft". We're trying to move away from that terminology for newer releases.

This adds a comma between `draft` and `date-draft` items and checks for
`empty` arrays to prevent stray "draft-0" output.
@davishmcclurg
Copy link
Contributor Author

It's on purpose that 2019-09 and 2020-12 don't say "draft". We're trying to move away from that terminology for newer releases.

Ah, that makes sense. I updated this to just add the comma and check for empty to prevent "draft-0" output.

Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to make the fix.

@Relequestual Relequestual merged commit a322db4 into json-schema-org:main Sep 8, 2023
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.

3 participants