Zadania wykonane - naniosłem wcześniejsze wskazówki#166
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Krystianie,
Całość wygląda bardzo dobrze! 👍
Zostawiłem parę drobnych uwag :)
| shape-outside: polygon(0 0, 50% 0, 0 50%); | ||
| margin: 0 16px 0 -16px; | ||
| line-height: 18px; | ||
| } |
| <link rel="stylesheet" href="./styles/mobile.css"> | ||
| <link rel="stylesheet" media="(min-width: 600px)" href="./styles/tablet.css"> | ||
| <link rel="stylesheet" media="(min-width: 1281px)" href="./styles/desktop.css"> |
There was a problem hiding this comment.
Twoje rozwiązanie działa odwrotnie do tego co zostało zapisane w zadaniu ;)
| grid-area: 1 / 2 / 2 / 3; /* Zaczyna się na 2 wierszu i 2 kolumnie, kończy się na 3 wierszu 3 kolumnie */ | ||
| width: 100%; | ||
| height: 100%; | ||
| } |
| order: 3; | ||
| width: 100%; | ||
| height: 30vh; | ||
| } No newline at end of file |
| grid-area: 2 / 1 / 3 / 2; /* Zaczyna się na 2 wierszu i 2 kolumnie, kończy się na 3 wierszu 3 kolumnie */ | ||
| width: 100%; | ||
| height: 100%; | ||
| } |
|
|
||
| /* Ukrycie */ | ||
|
|
||
| nav ul li ul { |
There was a problem hiding this comment.
Lepiej jest stylować po klasach - wtedy jest mniej problemów, spróbuj dołożyć jeszcze jedno zagnieżdżenie i zobaczysz co tam się może wydarzyć :P
|
|
||
| <!-- Moduł 1 --> | ||
|
|
||
| <section class="module--1 content row"> |
There was a problem hiding this comment.
Raczej staramy się używać nazw słownie zamiast samego --1 czy --2 (czasami trzeba zmienić ich kolejność i wtedy numerki nie będą po kolei, co może wprowadzać w błąd.
PS. Jeśli używamy BEM to zawsze Modyfikator występuje z Elementem lub Blockiem czyli tutaj powinno być tak: module module--1. Tutaj znajdziesz więcej informacji: https://devmentor.pl/b/metodologia-bem-w-css-i-sass
2 participants
Zadania wykonane - naniosłem wcześniejsze wskazówki