gohugoio / hugoThemesSiteBuilder

The source for https://themes.gohugo.io
https://themes.gohugo.io
Apache License 2.0
210 stars 190 forks source link

Add theme Quiet Foodie #465

Closed paposeco closed 3 months ago

paposeco commented 3 months ago

After you have read the instructions for adding a theme, please make sure:

netlify[bot] commented 3 months ago

Deploy Preview for hugothemes ready!

Name Link
Latest commit 7bfc763b1f4dc21c95dab24881810abe9cf8ed3f
Latest deploy log https://app.netlify.com/sites/hugothemes/deploys/664dcf71b60f20000849a842
Deploy Preview https://deploy-preview-465--hugothemes.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.

hugo-sid commented 3 months ago

Please use absolute paths for images in your README.

image

A purely optional suggestion

Your theme is available at github.com/paposeco/cleanhugotheme & you have named this PR as Add theme cleanhugotheme. So, I was under the impression that the name of your theme is cleanhugotheme.

But, upon seeing the theme's theme.toml, the name of the theme seems to be:

name = "Quiet Foodie"

To maintain consistency and avoid any confusion, it might be a good idea to align the name used in the repository and PR with the name specified in theme.toml. If "Quiet Foodie" is indeed the correct name, consider renaming the repository and updating the PR title to reflect this. This will help ensure clarity for anyone using or referencing your theme in the future.

Feel free to disregard if this was intentional or if there's a reason for this. Just a thought!

paposeco commented 3 months ago

Thanks for the feedback @hugo-sid! I didn't know you could change a repository name. ahah Much better this way. And the images should be fixed.