Basic HTML and CSS Project done#186
Basic HTML and CSS Project done#186TetraMeister wants to merge 11 commits intodevmentor-pl:masterfrom
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Łukaszu,
Projekt prezentuje się super! 👍
Zostawiłem parę drobnych uwag dot. semantyki :)
| &__text { | ||
| line-height: 2; | ||
| } | ||
| } No newline at end of file |
| } | ||
|
|
||
| &__link:hover { | ||
| color: #0ABCEB; |
There was a problem hiding this comment.
Warto kolory zapisywać w zmiennych - potem łatwiej je zmieniać jeśli zajdzie taka globalna potrzeba tj. klient: jednak wolę zielony od niebieskiego :D
index.html
Outdated
| <div class="container features__container"> | ||
| <div class="cell features__card"> |
There was a problem hiding this comment.
Tutaj mamy "listę" cech danego produktu - skoro lista to ja bym zrobił zamiast div x 2 to po prostu ul > li- będzie czytelniej w HTML (lepsza sementyka) - czyli każdy features__card to li
index.html
Outdated
| <section class="row about"> | ||
| <div class="cell about__img-container"></div> | ||
| <div class="cell about__content"> | ||
| <div class="about__content-container"> |
There was a problem hiding this comment.
Wrzuciłbym article bo mamy spójną treść, którą możemy przenieść w dowolne miejsce (nie musi być tej treści dużo).
index.html
Outdated
| <div class="card-section__container container"> | ||
| <div class="card-section__card cell pricing__card"> |
There was a problem hiding this comment.
Podobnie jak wcześniej, mamy "listę" pakietów wieć znów zrobiłbym ul > li
index.html
Outdated
| </div> | ||
| <div class="card-section__container container"> | ||
| <div class="card-section__card cell pricing__card"> | ||
| <div class="pricing__price-container"> |
There was a problem hiding this comment.
TUtaj można wrzucić header jako nagłówek dla listy pakietów. Ten znacznik nie tylko nadaje się do nagłówka całej strony.
index.html
Outdated
| <div class="card-section__container container"> | ||
| <div class="card-section__card cell team__card"> |
There was a problem hiding this comment.
Tutaj znów "lista" członków zespołu - wiesz co robić :P
|
Poprawiony HTML — faktycznie trochę poleciałem z tymi divami. Jakoś z automatu pisałem div, bo lepiej się zachowują przy stylowaniu — jest szybciej, zapomniałem, że reset.css resetuje za mnie niepotrzebne style. Dziękuje za poprawki. |
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Łukaszu,
Wszystkie zmiany są ok! 👍
| id="features" | ||
| > | ||
| <div class="container features__heading"> | ||
| <header class="container features__heading"> |
| <div class="cell features__card"> | ||
| </header> | ||
| <ul class="container features__container"> | ||
| <li class="cell features__card"> |
| fun | ||
| and good luck!</p> | ||
| </div> | ||
| </article> |
| </header> | ||
| <ul class="card-section__container container"> | ||
| <li class="card-section__card cell pricing__card"> | ||
| <header class="pricing__price-container"> |
| <div class="card-section__card cell team__card"> | ||
| </header> | ||
| <ul class="card-section__container container"> | ||
| <li class="card-section__card cell team__card"> |
2 participants
No description provided.