fix spacing in query-loop and post-title size in single/post templates

[madhusudhand]

This change fixes

• spacing around post title in homepage query loop.
• adjusts the post-title to h4 in query loop
• adjusts post title to h2 in single and post templates.

Before	After

Fixes: #190

[madhusudhand]

adjusts the post-title to h4 in query loop

@mikachan By referring designs h4 is the right size, but this changes seems to reverse the change done in #111.
I might be missing the context, can you confirm if it was intentional?

[mikachan]

By referring designs h4 is the right size, but this changes seems to reverse the change done in #111.

These titles were changed from h4 to h3 to maintain the correct heading hierarchy, as the 'Latest Posts' h3 heading was removed here.

We shouldn't really be changing the heading level just for styling reasons. Perhaps we need to reduce the font size of these headings via the query block settings instead?

[mikachan]

Thanks for this! The spacing change on the post title block looks good.

This PR has made me reconsider how we're using heading tags across the site. After looking at each template, I don't think we should be using a h1 for the site title in the header (it should just be a paragraph), and instead, ensure that each template has a contextual h1 (e.g. the query title on the archive page is a h1). What do you think?

[madhusudhand]

I don't think we should be using a h1 for the site title in the header (it should just be a paragraph), and instead, ensure that each template has a contextual h1

I Agree. It isn't contextually right to use h1 for Site title, it can just be a paragraph and bold font to be given by its class, where as for H1, the archive page title gives the right context.

[carolinan]

1+ for only having one H1 on each page. The home page also needs an H1 though.