Conversation
devmentor-pl
left a comment
There was a problem hiding this comment.
Marku,
Całość prezentuje się bardzo dobrze! 👍
Zostawiłem parę drobnych uwag :)
| <meta http-equiv="X-UA-Compatible" content="ie=edge"> | ||
| <title>devmentor.pl - HTML & CSS BASICS - #01</title> | ||
| <style> | ||
| /* Dodałem ostylowanie, żeby nie oślepnąć od ekranu */ |
|
|
||
| <article class="article"> | ||
| <h2>Lorem ipsum dolor</h2> | ||
| <img src="" alt="PICTURE"> |
There was a problem hiding this comment.
Można też dodać <figure> z odpowiednim opisem dla zdjęcia :)
| <div class="comment-history"> | ||
| <div class="comment" id="last-comment"> |
There was a problem hiding this comment.
Każdy komentarz może być article jako jedna spójna wypowiedź lub można zrobić ul > li jako że to LISTA komentarzy :)
| (0, 2, 2) | ||
| */ | ||
| ?? { | ||
| .ac-container input:checked ~ article { |
| @font-face { | ||
| font-family: ; | ||
| src: url(); | ||
| } No newline at end of file |
There was a problem hiding this comment.
Tutaj zabrakło podłączenia odpowiednich plików.
TUtaj cały artykuł jak to zrobić: https://www.smashingmagazine.com/2013/02/setting-weights-and-styles-at-font-face-declaration/
| <li class="nav__item">ITEMS | ||
| <i>></i> | ||
| <ul class="nav__dropdown"> | ||
| <li class="dropdown__item">Item 1</li> |
There was a problem hiding this comment.
Jeśli korzystamy z BEM to zawsze w hierarchii musi być Block (nawet jeśli go nie stylujemy) - tutaj zabrakło dropdown. Należy go dodać do ul powyżej tj. <ul class="nav__dropdown dropdown"> - czyli mamy dwie klasy i wszystko jest ok! :)
| } | ||
|
|
||
| .nav__dropdown-2 { | ||
| transform: translate(190px, -51px); |
There was a problem hiding this comment.
A co jeśli rodzić będzie zajmować więcej przestrzeni? Lepiej ustawić left: 100% i np. top: 0 - wtedy mamy bardziej uniwersalne rozwiązanie
|
|
||
|
|
||
| </body> | ||
| </html> No newline at end of file |
| .footer__item { | ||
| max-width: 50%; | ||
|
|
||
| } No newline at end of file |
No description provided.