Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sebastianie,
Projekt prezentuje się bardzo dobrze! 👍
Zostawiłem parę drobnych uwag - musisz bardziej popracować nad semantyką :)
| <div class="header__nav--menu" id="#" <a href="#"></a> | ||
| <a href="#team">The Team</a> | ||
| <a href="#pricing">Pricing</a> |
There was a problem hiding this comment.
Pamiętaj o semantyce - tutaj zdecydowanie pasuje nav, dodatkowo osobiscie wolą strukturę ul > li, niż samo a, ale to nie błąd.
| </div> | ||
| </nav> | ||
| <div class="header__front wrapper"> | ||
| <div class="header__front--text"> |
There was a problem hiding this comment.
TUtaj można wrzucić article :)
| <div class="header__front wrapper"> | ||
| <div class="header__front--img"></div> | ||
| </div> |
There was a problem hiding this comment.
Można też figure > img
PS. Pamiętaj, że Modyfikator z BEM nie występuje sam:
https://devmentor.pl/b/metodologia-bem-w-css-i-sass
| <div class="main__features--list wrapper"> | ||
| <div class="main__features--list-one display-flex"> |
There was a problem hiding this comment.
Tutaj również można użyć ul > li bo to lista cech więc ta struktura pasuje. Każde li to osobna cecha.
| <p>Up to 7 Projects</p> | ||
| <p>2 Additional Developers</p> |
There was a problem hiding this comment.
Tutaj jest lista cech dla konktretnej wersji cenowej więc powinno być ul > li
| </div> | ||
|
|
||
| <div class="pricing__offers wrapper"> | ||
| <div class="pricing__offer"> |
There was a problem hiding this comment.
Sporo jest div-ów więc zrobiłbym article dla każdego pakietu cenowego, bo to oddzielna, jasna spójna całość.
| href="https://fonts.googleapis.com/css2?family=Montserrat:ital,wght@0,100..900;1,100..900&display=swap" | ||
| rel="stylesheet" | ||
| /> | ||
| <link rel="stylesheet" href="../task-html-and-css-basics/style.css" /> |
There was a problem hiding this comment.
Raczej poprawny adres to ./style.css - ścieżki piszmy względem głównego katalogu, a "ponad".
Ta zmiana spowoduje, żę inne obrazki nie będą działać - trzeba by było je poprawić.
Raczej standardowo, pobierając repo to uruchamiamy jego zawartość względem tego katalogu z repo tj. task-html-and-css-basics - niestety w obecnej wersji to nie zadziała.
2 participants
No description provided.