Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sandro,
Strona prezentuje się bardzo dobrze! 👍
Zostawiłem jeszcze parę uwag dot. semantyki i nazywania klas.
Widać jednak postęp względem zadań - tak trzymaj!
index.html
Outdated
| <span>The Team</span> | ||
| <span>Pricing</span> | ||
| <span>Features</span> |
There was a problem hiding this comment.
Osobiście wolę tworzyć menu przy pomocy ul > li, to co zrobiłaś nie jest błędem - tak tylko daję znać.
PS. Pamiętaj tylko o <a> - bez tego nie ma linków :) do href można wstawić # jeśli nie ma odnośników
index.html
Outdated
| </nav> | ||
| </header> | ||
| <section class="container1-section2"> | ||
| <div class="section-left"> |
There was a problem hiding this comment.
Można też dać article - wystarczy spójna treść, nie musi być tego dużo.
| </p> | ||
| <button class="button--section-1">Download FREE!</button> | ||
| </div> | ||
| <div class="section-right"> |
index.html
Outdated
| <section class="container2-section2"> | ||
| <div class="items-features"> |
There was a problem hiding this comment.
Można też użyć ul > li, bo mamy "listę" cech
index.html
Outdated
| </section> | ||
| </div> | ||
| </div> | ||
| <div class="container section-3"> |
There was a problem hiding this comment.
Zrobiłbym aside, bo jest to coś dodatkowego / reklama.
index.html
Outdated
| <section> | ||
| <div class="container5-section2"> | ||
| <div class="container5-section2--items"> | ||
| <h5>BASIC</h5> |
There was a problem hiding this comment.
Chyba bym dał h4, bo dla sekcji nasz h3 - więc aby było po kolei (to też nie błąd, ale jakoś tak zasadniej)
index.html
Outdated
| </section> | ||
| </div> | ||
| </div> | ||
| <div class="container section-5"> |
There was a problem hiding this comment.
Skoro nazwa klasy to section, to może warto też użyć takiego znacznika?
index.html
Outdated
| <link href="https://fonts.googleapis.com/css2?family=Poppins:wght@400;600;700&display=swap" rel="stylesheet"> | ||
| </head> | ||
| <body> | ||
| <div class="container section-1"> |
There was a problem hiding this comment.
Numerowanie klas nie zawsze jest dobrym pomysłem. Co jeśli zmienimy ich kolejność? W kodzie będzie 3, 2, 4, 1, 5 - będzie to ok? Czy może będzie zmieniać nawy klas. Tak czy inaczej będzie to problematyczne ;)
index.html
Outdated
| <div class="container6-section2"> | ||
| <div class="container6-section2--items"> |
There was a problem hiding this comment.
TUtaj też można zrobic ul > li jako lista członków zespołu :)
index.html
Outdated
| <img src="./images/team-facebook.svg" alt="fb icon" class="icon-small"> | ||
| </div> | ||
| </div> | ||
| <div class="container6-section2--items"> |
There was a problem hiding this comment.
Jeśli chce Tworzyć nazwy klas zgodnie z BEM (tutaj szczegóły: https://devmentor.pl/b/metodologia-bem-w-css-i-sass) to razem z Modyfikatorem musi wystąpować Element czyli tutaj powinno być: container6-section2 container6-section2--items
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Sandro,
Dużo lepiej - tak trzymaj 👍
Zostawiłem jeszcze parę drobnych uwag, nie musisz ich nanosić - tylko następnym razem pamiętaj ;)
| </head> | ||
| <body> | ||
| <div class="container section-1"> | ||
| <div class="container section--header-and-template"> |
There was a problem hiding this comment.
Jeśli robisz Modyfikator, to musi być też nazwa, której dotyczy. Tutaj mamy Block wiec powinno być: class="container section section--header-and-template", gdzie section to style wspólne dla wszystkich tego typu bloków, a section--header-and-template zawiera tylko style dla tego konkretnego modyfikatora
| </header> | ||
| <section class="container1-section2"> | ||
| <div class="section-left"> | ||
| <article class="section-left"> |
| </div> | ||
| </div> | ||
| <div class="container section-2"> | ||
| <div class="container section--features"> |
There was a problem hiding this comment.
Podobnie jak wyżej, może lepszą nazwą by było po prostu: <div class="container container--features"> - wtedy nie trzeba tyle zmieniać :)
| <section> | ||
| <ul class="features-list"> | ||
| <li class="items-features"> | ||
| <div class="icon-container2"> |
| <section class="container4-section1"> | ||
| <figure> |
There was a problem hiding this comment.
Myślę, że wystarczyłoby samo figure - nie jest to błąd, ale wolę mniej skomplikowaną strukturę HTML :P
| <h5>BASIC</h5> | ||
| <ul class="container5-section2"> | ||
| <li class="container5-section2--items"> | ||
| <h4>BASIC</h4> |
| <p class="text-align text-bold"> | ||
| Up to 7 Projects<br> | ||
| 2 Additional Developers | ||
| </p> |
There was a problem hiding this comment.
Teraz zauważyłem, że tutaj też można zrobic ul > li, gdzie każda cecha danego pakietu to osobne li - w końcu to lista elementów, które wchodzą w skałd danego pakietu.
PS. Tak, nie ma problemu z tym, że zagnieżdżamy jedną listę w drugiej.
2 participants
No description provided.