<p> size and line height

consento-org / consento-website

Ghost theme used for consent.org domain

https://consento.org

Other

1 stars 0 forks source link

<p> size and line height #89

Closed Tannybunny closed 4 years ago

Tannybunny commented 4 years ago

Looking through the long texts on the website I realized the body text needs adjustment. I suggest making it overall a bit smaller and increasing the line height for better readability of long text. I found this might be suitable: font-size: 1.4em; line-height: 1.6em;

font size and line height

Torgian commented 4 years ago

Refer to 1ae76ca

This changes the paragraph element to your specifications. Note this affects the home page as well, not just blog posts. However, this does not affect the footer text.

The above push is live on the website now, so you can review it there too.

Do you want the footer text to match the rest of the style? Below are screenshots with and without your new changes.

With change:

@Tannybunny Please review

Tannybunny commented 4 years ago

I think overall it looks good, at least better :D I guess it's good to also apply it to the funding text, to distinguish it further from the footer links. There are some spacing issues though between "p" and "h2" .. should I open a separate issue for that?

Torgian commented 4 years ago

Modified footer text here: https://github.com/consento-org/consento-website/commit/d7de4ac58fafc89058ab5ef1e1a6b98bfcb19bcc