Finished my first project-task#172
Finished my first project-task#172code-mike-code wants to merge 1 commit intodevmentor-pl:masterfrom
Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Michale,
Całość prezentuje się bardzo dobrze! 👍
Zostawiłem parę komentarzy dot. znaczników - myślę, że można być, co do nich mniej konserwatywny i będzie idealnie ;)
| href="https://fonts.googleapis.com/css2?family=Montserrat:ital,wght@0,100..900;1,100..900&display=swap" | ||
| rel="stylesheet" | ||
| > | ||
| <!-- <link |
There was a problem hiding this comment.
Zastanawiam się czemu zakomentowane. Spokojnie można korzystać z reset.css i nie trzeba go komentować na późniejszym etapie prac ;)
There was a problem hiding this comment.
Cześć, problem pojawił się w momencie kiedy dodałem plik reset.css, wszystko mi się rozjechało
| <div class="features__content container"> | ||
| <div class="feature"> |
There was a problem hiding this comment.
Osobiście wrzuciłbym tutaj ul > li, gdzie li to "feature" bo mamy tutaj "listę" cech więc idealnie to pasuje ;P
|
|
||
| </section> | ||
|
|
||
| <section class="logos main-container"> |
There was a problem hiding this comment.
Może nawet aside, jako coś dodatkowego
| </div> | ||
|
|
||
| <section class="hero container"> | ||
| <div class="hero__info"> |
There was a problem hiding this comment.
Może nawet pokusiłbym się o article, bo mam jakąś spójną wypowiedź.
|
|
||
| </div> | ||
| </div> | ||
| <div class="about__info"> |
There was a problem hiding this comment.
Tutaj również może article jako spójna wypowiedź.
| id="offer" | ||
| class="offer main-container" | ||
| > | ||
| <div class="offer__heading container"> |
There was a problem hiding this comment.
Tutaj wrzuciłbym header - możemy użyć tego znacznika jako nagłówek całej strony, ale również nagłówek do sekcji.
| <div class="offer__plans container"> | ||
| <div class="offer__card offer__card--shadow"> |
There was a problem hiding this comment.
Mamy tutaj "listę" pakietów cenowych więc ponownie użyłbym ul > li, gdzie li to każdy z pakietu :)
2 participants
No description provided.