my proposition

[JustynaJBroniszewska]

No description provided.

[IamMK]

Ok. Więc zaczynamy.
Nie podobają mi się nazwy klas w HTML.

<button type="submit" id="btn" class="btn-primary">Get early access </button>
<input type="email" requider placeholder=" " id="username" class="my_input">
<img id="img-here" src="assets/undraw_developer_activity_bv83.svg" width=185 height=153 alt="">

Nie wiem czy po tych nazwach by było można określić choćby na oko gdzie co jest ;)

<main class="grid">
<section id="section-a" class="grid">
 <footer id="main-footer" class="grid">

Po co tyle tego grida? I jeśli nie używasz js na danym elemencie niekonieczne musi być używanie id :)
Stylować najlepiej po css, w razie gdyby element miał być wykorzystany ponownie gdzieś w kodzie nie musisz nic przerabiać w kodzie ;)

<h1>User &copy 2019<br>
                Made with <i class="fas fa-heart"></i> in Poland <br>
                We're using cookies to improve your<br> experience.
</h1>

Tutaj wielkie, wielkie ała... H1 to nie jest header w który wpierdzielasz całą sekcję. Służy przede wszystkim jako tytuł sekcji. Tutaj jest jakby... zbędny? Zupełnie jak tagi <br />. Odchodzi się od nich. Jeśli możesz(a rzadko Cię coś ogranicza) używaj tagu <p>.

</footer>
</main>

footer miał być w main czy niekoniecznie?

margin-left: auto;
margin-right: auto;

a nie lepiej

margin: auto;

figcaption {}
img {}
input,
.btn-primary{}

Staraj się praktykować stylowanie po klasach. Na pewno się nie zemści ;)
co do ostatniego. Nie lepiej zrobić jedną klasę dla obu elementów?

#btn

jak wcześniej wspomniałem, staraj się stylować po klasach. Większa elastyczność użycia 😄

[PatrykPorabik]

<br />.

@IamMK <br> :D

Osobiście mi się nie podoba to: width="124" height="189" w div'ie z obrazkiem.

[JustynaJBroniszewska]

:) Dziękuję pięknie za wszystkie uwagi, mam nadzieję, że następnym razem będzie lepiej ;)