-
-
Notifications
You must be signed in to change notification settings - Fork 244
New Theme : Hugo Lime #413
Comments
Hello @heyarviind, thank you for submitting another beautiful theme to us 👍 I've tested your theme with the build script for the Hugo theme site that also generates a demo for the theme. When you make use of the exampleSite folder please make sure that
|
Hey @digitalcraftsman, Sorry for the mistakes, I've resolved the issues. Please have a look at the repo again Cheers |
While having another looking at your theme I noticed that you're making us of Hugo's SCSS generation feature. Currently, the extended version of Hugo, which includes the SCSS generator, isn't supported by our hoster netlify.com. They're using a tool to fetch the latest Hugo release, but it can't distinguish between the normal Hugo version and the extended version of Hugo. A pull request to fix this problem has already been submitted. But it can take some time once Netlify also updates their build image that is used for the generation deployments. According to this comment a possible workaround is to let Git track the |
@digitalcraftsman I can compile and use the CSS for the demo if you say? |
For now you've to compile the CSS files yourself and add resources/_gen/assets that contains the generated to the Git repository. Because the extended version of Hugo isn't available during the build of the theme site Hugo is unable to compile the SCSS files on it's own. |
@digitalcraftsman I've added compiled CSS, please let me know if there is anything to be corrected |
@digitalcraftsman I am waiting for the theme to be added on hugo directory.... is there anything left from my side? |
@heyarviind You need to commit the Also please review the updated README Please do the above and let me know. Thanks. |
@onedrawingperday I tried build script but something was not working on my end, so I've added resource folder and changed files as you've mentioned. Please have a look |
First of all please delete the Then please review the Hugo Theme's updated README Common Permalink Issues because your demo is affected. You need to use Please do the above and let me know. |
@onedrawingperday Sorry for the I've also replaced |
@heyarviind Thanks for making the changes. All is good with your theme now and there are no more errors on my end. However I am not sure about what is the practice for the BUY NOW button that you have included in your header and also about the extra features of the PRO VERSION of your theme, that are mentioned in the README. The license of the free version of your theme is the standard MIT. So I am tagging @digitalcraftsman and @bep to offer their opinion about the promotion of the PRO VERSION of your theme. Also here is the link of your theme's repo to make it easier for them to see it for themselves: |
@onedrawingperday I develop free and pro version of the themes, so I am just submitting the free version to Hugo. I think adding little about pro version in the description has no harm. After all, it encourages me to develop more and professional looking themes for Hugo |
Sure, but this is not my call. I tagged the others to have a look at your theme's README about the features of the PRO VERSION as well as the BUY NOW button. If @bep and @digitalcraftsman are fine your theme can be added as is. If they request changes about the promotion, they will let you know. |
Also I want to note that in your previous theme submission that I also reviewed the promotion of the PRO version was more subtle and you only mentioned it in the README. There is no BUY NOW button in the Demo. |
Hello @heyarviind, your theme's demo is working fine. Regarding the promotion of a paid version I would refer to a similar discussion for Themefisher themes (e.g. navigator, airspace), especially this comment by Bjørn Erik. The MIT license is standard license and commonly. For me the README is formulated neutral. You're listing the differences between the free and pro version of the theme and linking to the pro version in a non-instrusive manner. Hence it should be (IMO) in accordance to Bjørn Erik's linked comment from above. However, I'm not sure about the "Buy now" button in the demo. This sounds like the theme isn't free which stands in contrast to the MIT license. A more neutral version could be "Show pro version". However, I would like to hear @bep's opinion as well for a final decision. /ping @bep |
I agree with you @digitalcraftsman -- "buy now" is hinting that this is a "try, then buy" kind of program. Which is fine, but not here. |
@digitalcraftsman @onedrawingperday @bep I've removed "buy now" button from the demo, please have a look. I don't think there is any other issue left with this theme |
Now I'm fine with this decision and your theme in general. What about you @bep? |
Looks good. |
Your theme should appear soon on Hugo's theme site. Next, I'll promote your theme on Hugo's official Twitter account. |
Theme submission
Before submitting a theme please make sure you've read the README.md. It describes in detail what information are needed and which requirements have to be fulfilled for the submission. The check list below can help you.
https://github.com/uicardiodev/hugo-lime
I made sure that...
README.md
describing my themeLICENSE.md
theme.toml
images/
folder with the required dimensionshttps://example.com
is set as base url inexampleSite/config.{toml, yaml, json}
to avoid the abuse of unused domainstoCSS
andPostCSS
that I have committed the/resources
directory with all generated assets, for my theme to work in the basic version of HugoFeel free to ask questions. We're glad to help.
The text was updated successfully, but these errors were encountered: