Skip to content

Update docs: Add more details about templates #103

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tobiase
Copy link

@tobiase tobiase commented Jan 26, 2025

  • Repository URL
  • Last commit date badge

Copy link

netlify bot commented Jan 26, 2025

Deploy Preview for decap-www ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/decap-www/deploys/679742b47bdbe207e96c7e48
😎 Deploy Preview https://deploy-preview-103--decap-www.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.

@tobiase tobiase changed the title Add more details about templates Update docs: Add more details about templates Jan 26, 2025
@martinjagodic
Copy link
Member

@tobiase, thanks for your submission. This section needs a refactor, would you be willing to take a dig? I investigated a bit and think we need to add MDX capabilities to our templates. I would do it myself, but my Gatsby game is weak.

I also have a style comment: the color of the badge depends on the commit date, which makes the page too colorful for my taste. Can you add a parameter to the badge link that sets the same color for all badges? I was thinking ?style=flat-square&color=lightgray. And some gaps between grid rows would be nice too.

@tobiase
Copy link
Author

tobiase commented Jan 28, 2025

Thanks for the feedback and suggestions!

I’m happy to take a dig at the refactor. I’ve already spent about an hour exploring it, but I’m not sure if I can fully implement it since I also have limited experience with Gatsby. I’ll give it another shot, though. If I hit a roadblock, I’ll let you know.

Regarding your style suggestions, I completely agree. I should have previewed the changes instead of committing directly via the GitHub UI—lesson learned for next time!

@martinjagodic
Copy link
Member

Thanks for the effort! If you can't find a way to add MDX, a good alternative would be to externalize the styles to make the HTML more readable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants