Don't output METAL tag in rendered Chameleon output.

Pylons / pyramid-cookiecutter-starter

A Cookiecutter (project template) for creating a Pyramid starter project with choices for template language (Jinja2, Chameleon, or Mako), persistent backend (none, SQLAlchemy with SQLite, or ZODB), and mapping of URLs to routes (URL dispatch or traversal)

95 stars 53 forks source link

Don't output METAL tag in rendered Chameleon output. #124

Closed mcdonc closed 1 year ago

mcdonc commented 1 year ago

If metal:define-macro is present on DOCTYPE, the rendered version of the template retains the literal metal attribute, which causes browsers to be unhappy. Moving it to the html tag serves the same purpose, but the rendered template does not contain a literal metal tag.

metalpyramidscaffold

mcdonc commented 1 year ago

I am also going to merge this, as it's so trivial and breaks nothing, but just FYI, I will submit pull requests and ask for a review on anything more complex.

mmerickel commented 1 year ago

Awesome thanks @mcdonc. If you want to full release it then probably need to apply this to latest and one of the release branches as well.