i think Urban Harvest needs <h1> tags rather than <p>? This is more semantic and knowing which text is headings (and what level those headings are) helps search engines index pages properly.
GREAT to see SVG animation being used!
Consider look into BEM - e.g. your harvest links could be rewritten:
<section class="harvest-links">
<button class="harvest-links__add">Add a discovery</button>
<button class="harvest-links__search">Search the field</button>
<button class="harvest-links__all">See all</button>
</section>
This is a small change but once you get used to the more clunky appearance it makes your HTML and CSS files easier to read and navigate.
I'll just put all this in one place...
<h1>
tags rather than<p>
? This is more semantic and knowing which text is headings (and what level those headings are) helps search engines index pages properly.This is a small change but once you get used to the more clunky appearance it makes your HTML and CSS files easier to read and navigate.