Tasks completed#172
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Arturze,
Zadania są ok 👍
Zdecydowanie zaleciłbym używanie klas nad zagnieżdżaniem nazwy tagów.
Zwróć uwagę również na zadanie 02, gdzie kolejność elementów się nie zgadza - warto przed oddaniem sprawdzić jeszcze raz wszystko (dobry nawyk na rekrutacji).
| border-right-color: transparent; | ||
| border-bottom-color: transparent; | ||
| shape-outside: polygon(0 0, 0% 100%, 100% 0); | ||
| } |
| grid-template-areas: | ||
| 'A A' | ||
| 'C B'; | ||
| grid-template-columns: 1fr 1fr; |
There was a problem hiding this comment.
Zwróć uwagę, że nie zgadza się kolejność na mobilce ;)
| title="YouTube video player" frameborder="0" | ||
| allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture; web-share" | ||
| referrerpolicy="strict-origin-when-cross-origin" allowfullscreen></iframe> | ||
| </side> |
| 't t'; | ||
| max-width: 1400px; | ||
| margin: 0 auto; | ||
| } |
| top: 0; | ||
| width: 100%; | ||
| height: 100%; | ||
| } |
| 'p t' | ||
| 'v t'; | ||
| grid-template-columns: minmax(200px, 50%) auto; | ||
| } |
| grid-template-columns: auto 1fr; | ||
| } | ||
|
|
||
| nav label { |
There was a problem hiding this comment.
Pamiętaj, że lepiej jest stylować po klasach (nie musi to być BEM).
Ten sposób nie jest problematyczny jak masz tylko jeden element, ale jak masz wiele podobnych elelemntów na stronie to zdecydowanie utrudniasz sobie działania ;)
| display: block; | ||
| } | ||
|
|
||
| nav ul > li ul { |
There was a problem hiding this comment.
Zdecydowanie łatwiej i czytelniej użyć odpowiedniej klasy :P
| /* SECTION TWO */ | ||
| .section-two .wrapper { | ||
| grid-template-columns: 1fr 1fr 1fr; | ||
| } |
|
|
||
| footer .content { | ||
| max-width: 200px; | ||
| } |
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Arturze,
Zmiany są ok, ale w przypadku zdania 04 raczej nie wystarczające :P
| } | ||
|
|
||
| nav label { | ||
| .navigation label { |
There was a problem hiding this comment.
Jeśli BEM to tutaj również klasa powinna być (wystarczy ta klasa dla label)
PS. Podobnie poniżej.
| 'A' | ||
| 'B' | ||
| 'C'; | ||
| } |
2 participants
No description provided.