Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Piotrze,
Projekt prezentuje się bardzo dobrze! 👍
Nie mam dużo uwag - parę drobnych :)
| <div class="image"> | ||
| <img src="/images/imac.png" alt="iMac" /> | ||
| </div> | ||
| <div class="text"> |
There was a problem hiding this comment.
Tutaj wrzuciłbym znacznik article zamiast div - bo to faktycznie jakaś konkretna treść.
| <p class="text">Up to 7 Projects</p> | ||
| <p class="text">2 Additional Developers</p> |
There was a problem hiding this comment.
Tutaj również zrobiłbym ul > li bo to lista cech dla tego pakietu cenowego
| <link rel="stylesheet" href="style/style.css"> | ||
| </head> | ||
| <body> | ||
| <section class="main"> |
There was a problem hiding this comment.
Zwróć uwagę, że na grafice jest jeden gradient, a nie dwa jak u Ciebie.
Należałoby dodać jeszcze jedne element nadrzędny dla .main oraz .main-content i temu elementowi nadać odpowiedni gradient- wtedy będzie on jeden dla oby elementów.
| <button>Download FREE!</button> | ||
| </div> | ||
| <div class="main__image"> | ||
| <img src="/images/screen.png" alt="Next Gen App"> |
There was a problem hiding this comment.
Zwróć uwagę, że te obrazek ma obramowanie na grafice ;)
| <h2>Choose Your Pricing Plan</h2> | ||
| <p>Pick any of our super affordable pricing plans</p> | ||
| <ul class="pricing-container"> | ||
| <li class="pricing-item"> |
There was a problem hiding this comment.
Brakuje mi odstępów między pakietami cenowymi tak jak to jest na grafice.
| color: #fff; | ||
| } | ||
|
|
||
| .main__content h1 { |
There was a problem hiding this comment.
Lepiej stylować tylko po klasach czyli należałoby nadać odpowiednią klasę dla <h1> i tutaj jej tylko użyć :)
2 participants
Project