feat: Create Nova Template Layout#177
feat: Create Nova Template Layout#177zanetasochon wants to merge 3 commits intodevmentor-pl:masterfrom
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Żaneto,
Projekt prezentuje się bardzo dobrze! 👍
Parę uwag dot. semantyki czy BEM zostawiłem w komentarzach :)
index.html
Outdated
| </ul> | ||
| </nav> | ||
| <section class="container__nav hero"> | ||
| <div class="left-section__hero"> |
There was a problem hiding this comment.
Zrobił bym z tego article, niby mała treść, ale spójna więc kwalifikuje się na ten znacznik :)
index.html
Outdated
| </a> | ||
| </div> | ||
| </div> | ||
| <div class="ipad-screen__hero"> |
index.html
Outdated
| <div class="features__offer"> | ||
| <div class="offer__item"> |
There was a problem hiding this comment.
Może ul > li bo to lista elementów, nawet nazwy klas o tym świadczą.
PS. W BEM jeśli występuje Element tj. offer__item, to musi występować Block tj. offer - nawet jeśli nie jest on stylowany
| </section> | ||
|
|
||
| <section class="about__container"> | ||
| <figure class="imac-screen"> |
There was a problem hiding this comment.
Dlaczego wiec w pozostałych nie użyłaś tego znacznika?
There was a problem hiding this comment.
Trochę się miotam z BEM i znacznikami, w trakcie pisania HTMLa dopiero gdzieś wygooglowałam co to jest figure, bo mi się divoza zrobiła, ale w docsach MDN jest napisane, że "referenced as a single unit" co rozumiem, że jest to element pojedynczy, ale co to znaczy? bez treści? I umieściłam go testowo :D
There was a problem hiding this comment.
Myślę, że tutaj znajdziesz odpowiedź: https://stackoverflow.com/questions/37828046/what-is-the-purpose-of-the-figure-tag-in-html
Jeśli obrazek (i nie tylko) reprezentuje przekaz tj. niesie za sobą treść to wtedy używamy figure.
index.html
Outdated
| <figure class="imac-screen"> | ||
| <img src="./images/imac.png" alt="White Imac on light background" /> | ||
| </figure> | ||
| <div class="about__section-text"> |
There was a problem hiding this comment.
Czy article nie służy do artykułów? Posty na blogu itd?
index.html
Outdated
| </section> | ||
|
|
||
| <section class="plan__container"> | ||
| <div class="plan__header"> |
There was a problem hiding this comment.
Może header jako znacznik dla section. To nie tylko element dla nagłówka całej strony, ale i artykułu czy sekcji.
There was a problem hiding this comment.
Good to know!
index.html
Outdated
| <div class="plan__cards"> | ||
| <div class="plan__card-item"> |
There was a problem hiding this comment.
Tutaj też można by było ul > li
| <section class="form__container"> | ||
| <h1 class="form__title">Download FREE!</h1> | ||
| <p class="form__description">Take it for a ride for any of your projects!</p> | ||
| <form method="post" action="template.php" class="email__form"> |
There was a problem hiding this comment.
Na samej grafice tego już nie było :P
There was a problem hiding this comment.
Faktycznie, dopiero zwróciłam uwagę! Wzorowałam się na live preview, łatwiej mi tak jak można sobie "pochodzić" po stronie, zobaczyć efekty, choćby przypomnieć sobie linkowanie po ID na onepage żeby po kliknięciu nam się ładnie sunęło do sekcji. Podlinkowałeś w Readme link do projektu i założyłam z góry, że warto, czy to jest ok? :)
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Żaneto,
Wszystko jest ok. Odpowiedziałem również na komentarz :)
| </div> | ||
| <div class="ipad-screen__hero"> | ||
| </article> | ||
| <figure class="ipad-screen__hero"> |
| <div class="features__offer"> | ||
| <div class="offer__item"> | ||
| <ul class="offer"> | ||
| <li class="offer__item"> |
| </li> | ||
|
|
||
| <div class="offer__item"> | ||
| <li class="offer__item"> |
| </section> | ||
|
|
||
| <section class="about__container"> | ||
| <figure class="imac-screen"> |
There was a problem hiding this comment.
Myślę, że tutaj znajdziesz odpowiedź: https://stackoverflow.com/questions/37828046/what-is-the-purpose-of-the-figure-tag-in-html
Jeśli obrazek (i nie tylko) reprezentuje przekaz tj. niesie za sobą treść to wtedy używamy figure.
2 participants
No description provided.