Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

Theme submission - Tandoori Hugo Theme #796

Closed
10 tasks done
Gnappuraz opened this issue Jan 26, 2020 · 4 comments
Closed
10 tasks done

Theme submission - Tandoori Hugo Theme #796

Gnappuraz opened this issue Jan 26, 2020 · 4 comments
Labels

Comments

@Gnappuraz
Copy link

Theme submission

Link to my theme repository: https://github.com/Gnappuraz/tandoori-hugo

I made sure that...

  • the repository contains a good README.md describing my theme
  • an open source license has been added to LICENSE.md
  • all metadata have been added to theme.toml
  • screenshots have been added in the images/ folder with the required dimensions
  • in case I'm using a customized demo via the exampleSite folder that
  • I tested my theme against the gohugoio/HugoBasicExample
    • I've checked the developer tools' console in my browser for error messages
  • in case my theme is using Hugo Pipes features like toCSS and PostCSS that I have committed the /resources directory with all generated assets, for my theme to work in the basic version of Hugo
@onedrawingperday
Copy link
Contributor

onedrawingperday commented Jan 27, 2020

Hello @Gnappuraz

Thank you for submitting this fork of the Beautiful Hugo theme.

However you need to list the new features that you have developed on top of the original theme in the README. Currently it is not very clear.

Also I have seen that you mention pygments configuration parameters in the README. However pygments has been removed from Hugo since v0.60.0 and these parameters will no longer have any effect since the new default is Chroma and code highlighting is also enabled by default. See: https://gohugo.io/content-management/syntax-highlighting/#highlighting-in-code-fences

Once you look into the above please let me know so that I can review your theme further.

@Gnappuraz
Copy link
Author

Gnappuraz commented Feb 2, 2020

Hi @onedrawingperday, sorry for the late response but it has been a hell of a week ;). to answer your concerns:

  • I removed support for everything but the default Chroma (this way is easier).
  • I made clear that this fork is just a restyle, I actually did not develop any new feature, for the most part I just fiddled with the css.
  • I increased the min supported version to v0.60.0

@onedrawingperday
Copy link
Contributor

onedrawingperday commented Feb 4, 2020

@Gnappuraz

I have noticed the following issues while reviewing your theme:

Please change line 71 of the exampleSite/config.toml to url = "post/math-typesetting" and line 77 to url = "post/markdown-syntax"

Once you look into the above please let me know.

@stale
Copy link

stale bot commented Feb 25, 2020

This issue has been automatically marked as stale because it has not had recent activity. Feel free to ask questions. We're glad to help.
This issue will automatically be closed in the near future if no further activity occurs. Thank you for all your contributions.

@stale stale bot added the Stale label Feb 25, 2020
@stale stale bot closed this as completed Mar 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants