Responsive landing page using CSS Grid

Community feedback

• 

  Karol Binkowski•1,640

  @GrzywNPosted over 2 years ago

  Cześć, dobra robota!

  Jeśli chodzi o HTML, to myślę, że jest jak najbardziej prawidłowo. Osobiście nie lubię używać skróconych nazw typu 'btn', ale to już kwestia konwencji, jaką się przyjmie. Podsyłam fajny artykuł o nazywaniu klas link. Dobrze jest też podpiąć selektory JSa albo do klas typowo pod JS-a, ID albo atrybutów dataset. Chodzi o oddzielenie CSS-a i JS-a. Wszystko jest wtedy czytelniejsze i jeśli pracuje się na CSS-ie, to wtedy po zmianie nazwy nic się nie popsuję w funkcjonalności.

  Nie robiłem tego challenge'u, ale wydaje mi się, że aby przycisk był zrobiony poprawnie, to trzeba go przesunąć w prawo transformem o tyle, ile jest ustawiony border na polu input.

  Ostatnimi czasy zacząłem wszystkie elementy z DOM-u / querySelectory wrzucać do obiektu i myślę, że tutaj też by się to świetnie sprawdziło, ale to też kwestia preferencji. Patrząc na obiekt np. domElements.input wiadomo od razu, co oznacza i do czego się odnosi.

  Mam nadzieję, że choć trochę pomogłem :) Miłego dnia i kodowania!

  Marked as helpful

  0

  ---

  

  Joanna•370

  @JoannaLapaPosted over 2 years ago

  Cześć Karol,

  Dziękuję za Twój feedback, na pewno skorzystam z Twoich porad i podesłanych materiałów. Zdecydowanie pomogłeś :).

  Miłego dnia i owocnego kodowania!

  1