search

plone / plone.app.theming

Integrates the Diazo theming engine with Plone
https://docs.plone.org/external/plone.app.theming/docs/index.html
Other
21 stars 30 forks source link

documentation is incorrect: a barebones theme needs manifest.cfg #105

Open tkimnguyen opened 8 years ago

tkimnguyen commented 8 years ago

Re: https://github.com/plone/Products.CMFPlone/issues/1622 the documentation at http://docs.plone.org/external/plone.app.theming/docs/index.html#creating-a-new-theme says that uploading a ZIP containing just an index.html will work, because the rules.xml will get auto-generated. The theme does get created, however, without a manifest.cfg, the theme does not show in the @@theming-controlpanel.

The doc here should state that you need at least a minimal manifest.cfg containing 

[theme]
title = My New Theme
davilima6 commented 8 years ago

Actually documentation is right as we can confirm by looking at the changelog:

1.0b4 - 2011-05-24
Fix zip import to work correctly when no manifest is supplied. [elro]

We could improve docs by making that explicit.