search

thombruce / tnt

Thom's Nuxt Template
https://tnt.thombruce.com/
MIT License
1 stars 0 forks source link

[Feature]: Consider removing .prose from default layout #76

Closed thombruce closed 3 months ago

thombruce commented 3 months ago

Feature request

If default layout is intended to be general purpose, if it is intended to support general applications rather than just blog sites... should it not not use .prose?

The reason it does feature the .prose class is because, like all of the layouts, it was originally written for my blog and included as a layout on TNT Content (not Core). But it now lives on Core. I now use it to build Toodles.

I do still use the default layout for some pages of my personal site however, so this may be something we want to amend...

Should TNT Content have a different default? Should it overwrite the default layout present on Core?

Not sure. But as we move towards supporting apps built with TNT, we should think more carefully about that.

Code of Conduct

thombruce commented 3 months ago

Consider replacing <article></article> with <div></div> too, in fact. There is no reason to believe that the end user will be posting an article on every page.

Alternatively: Move main from app.vue to templates as the root element on template pages.

Generally reconsider the HTML structure a little bit and what our likely use cases are.

thombruce commented 3 months ago

Done: 99bce3e7627b3205c55d4a75b423807264aae91a

Due a refactor in the near future.