Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Marku,
Całość prezentuje się bardzo dobrze! 👍
Zostawiłem parę drobnych uwag ;)
| </head> | ||
| <body> | ||
| <section class="Main"> <!-- SECTION ------------------------------------------> | ||
| <header class="Main__header"> |
There was a problem hiding this comment.
Jeśli to BEM to nie zaczynamy od wielkiej litery - wszystko z małej
| <li class="Main__header__menu__item"> | ||
| <a class="Main__header__menu__item__link" href="#">The team</a> | ||
| </li> | ||
| <li class="Main__header__menu__item"> |
There was a problem hiding this comment.
Jeśli to BEM to nie możemy w nieskończoność robić ... _ ... _ ... _ mamy tylko 2 zagłębienia tj. BLOCK__ELEMENT--MODIFIER - koniec ;P
| </ul> | ||
| </header> | ||
| <section class="Main__content"> | ||
| <div class="Main__content__left"> |
There was a problem hiding this comment.
Można też wstatwić article zamiast div
| <div class="Features__header__box"> | ||
| <div class="Features__header__box__card"> |
There was a problem hiding this comment.
Można też zrobić ul > li bo to lista "cech"
| <p class="Price__box__card__text"> | ||
| Up to 7 Projects <br> | ||
| 2 Additional Developers <br> | ||
| </p> |
There was a problem hiding this comment.
Tutaj zdecydowanie użyłbym ul > li - mamy listę cechy danego pakietu
| <P class="Price__header__text">Pick any of our super affordable pricings</P> | ||
| </div> | ||
| <div class="Price__box"> | ||
| <div class="Price__box__card"> |
There was a problem hiding this comment.
Każdy z pakietów może też być article bo to jedna spójna całość
2 participants
No description provided.