Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Krystianie,
Odwzorowanie strony bardzo dobre! 👍
Semantyka do poprawy, zdecydowanie za dużo używasz article - postaraj się to poprawić zgodnie z wytycznymi w komentarzach :)
|
|
||
| <body> | ||
|
|
||
| <!-- Pytanie czy w VSC moge tworzyć własne szablony i donieśc się do nich jak w Emmet? --> |
There was a problem hiding this comment.
W zadaniach podrzuciłem Ci link ;)
| </article> | ||
| <article class="content__article cell right"> |
There was a problem hiding this comment.
Na pewno nie robiłbym 2 razy article. Bo ten tag dla samego img to za mało.
Zrobiłbym odwrotnie tj. div zamieniłbym na article, a div dodał do treści artykułu, aby móc go wypozycjonować. Natomiast zamiast okalać img przez article, to zrobiłbym figure.
PS. Tak title jest wykorzystywany jedynie w head. Tutaj powinno być h1 lub inny poziom
| <!-- Features --> | ||
|
|
||
| <section data-id="Features" class="content row"> | ||
| <div class="content__container container"> |
There was a problem hiding this comment.
Mozna zamiast div użyć header jako nagłówek dla sekcji
| <div class="content__container container"> | ||
| <article class="content__article cell"> |
There was a problem hiding this comment.
Osobiście zrobiłbym ul > li zamiast div > article, mamy listę cech więc lepiej się mi to komponuje.
|
|
||
| <section data-id="Module cooperating" class="content row"> | ||
|
|
||
| <article class="content__article cell"> |
There was a problem hiding this comment.
Zdecydowanie to nie article - mam wrażenie, że jak kiedyś się nadużywało div to ty teraz to wszystkiego starasz się użyć article. TO nie jest dobre rozwiązanie. Tutaj zdecydowanie lepioej będzie pasować aside.
| <div class="content__container container"> | ||
| <article class="content__article cell"> | ||
| <div> | ||
| <title>basic</title> |
There was a problem hiding this comment.
Ponownie zwracam uwagę, że tutaj raczej np. h2 lub h3
| </div> | ||
| <button>Get Started</button> | ||
| </article> | ||
| <article class="content__article cell"> |
|
|
||
| <section data-id="The team" class="content row"> | ||
| <div class="content__container container"> | ||
| <article class="content__article title--module">THE TEAM</article> |
There was a problem hiding this comment.
Zdecydowanie nie article, można użyć h3.
| <article class="content__article cell"> | ||
| <div> | ||
| <img class="person" src="images/cto.png" alt=""> | ||
| <title>CEO</title> |
| <div class="content__container container"> | ||
| <article class="content__article cell"> |
There was a problem hiding this comment.
Tutaj mamy listę członków zespołu więc ponownie zrobiłbym ul > li zamiast div > article
2 participants
Po hover w sekcji price jest animacja zgodna z makietą.