[z3-five] Re: r5538 - z3/Five/branch/dreamcatcher-cmf-compat-branch

[Philipp von Weitershausen]

Sidnei,

> +def skinDirectory(_context, directory, module, for_=None,
> +                  layer='default', permission='zope.Public'):

...

> +class ISkinDirectoryDirective(Interface):
> +    """Register each file in a skin directory as a resource
> +    """

I know that it is (sadly) common practice in the CMF world to call skin 
layers simply 'skins', but Zope 3 requires a good understanding of the 
two separate terms, otherwise people won't understand what's going on 
and think that Zope 3 does it all different, but it doesn't.

What you really mean here is a _skin layer_, so I would recommend 
calling this directive five:skinLayerDirectory or simply 
five:layerDirectory...

Btw, the docstring of the directive schema is incorrect. Currently, all 
the handler does is register every page template file as a browser 
page... ;)

Philipp