HTML & CSS Basics practice - Done#225
HTML & CSS Basics practice - Done#225TetraMeister wants to merge 10 commits intodevmentor-pl:masterfrom
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Łukaszu,
Całość prezentuje się bardzo dobrze! 👍
Zostawiłem jedną drobną uwagę do BEM oraz do nazwy klas - to drugie może wynikać z "prostoty" zadania, ale wolałem zwrócić na to uwagę :)
| </footer> | ||
|
|
| -o-transition: height 0.5s ease-in-out, box-shadow 0.1s linear; | ||
| -ms-transition: height 0.5s ease-in-out, box-shadow 0.1s linear; | ||
| transition: height 0.5s ease-in-out, box-shadow 0.1s linear; | ||
| box-shadow: 0 0 0 1px rgba(155, 155, 155, 0.3); |
| .parent--flexbox { | ||
| display: flex; | ||
| gap: 10px; | ||
| } No newline at end of file |
| .dropdown-menu__list-item:hover > a, | ||
| .dropdown-menu__sublist-item:hover > a { | ||
| color: white; | ||
| } No newline at end of file |
| <li class="nav__list-item"><a href="#">Home</a></li> | ||
| <li class="nav__list-item"> | ||
| <a href="#">Items</a> | ||
| <ul class="dropdown-menu__list"> |
There was a problem hiding this comment.
Jeśli mamy BEM i mamy Element o nazwie "dropdown-menu__list", to gdzieś w strukturze powinniśmy mieć Block o nazwie "dropdown-menu". U Cibie go brakuje - można dodać do najbliższego li.
Tutaj przeczytasz więcej o BEM: https://devmentor.pl/b/metodologia-bem-w-css-i-sass
| .footer__div { | ||
| height: 90%; | ||
| width: 250px; | ||
| } No newline at end of file |
|
|
||
| <section class="first-section"> | ||
| <article class="first-section__article"> | ||
| <div class="first-section__div"></div> |
There was a problem hiding this comment.
Umieszczenie w klasie nazwy tagu nie jest najlepszym pomysłem. Często tagi ulegają zmianie i potem trzeba też nazwę klasy zmieniać. Ogólnie w programowaniu zasada jest taka, że zmiana w jednym miejscu nie powinna wymuszać zmiany w innym.
There was a problem hiding this comment.
Dzielenie kodu CSS na pliki w późniejszych materiałach będzie "wygodniejsze", obecnie brakuje nam narzędzi, ale w niedalekiej przyszłości już tak nie będzie ;)
2 participants
W Task05 zrobilem troche inaczej jak w PDF potem poprawilem dlatego sa 2 proby (pierwsza nastapila przed przeczytaniem koncowki PDFa)