Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Dagmaro,
Całość prezentuje się ok! 👍
Zostawiłem parę drobnych uwag.
Zwróć uwagę proszę na formatowanie kodu oraz na większe wykorzystanie innych znaczników niż div` :)
There was a problem hiding this comment.
Dlaczego ten plik jest w katalogu images? Chyba przypadkowo go tam przerzuciłaś, bo domyślnie powinien być w katalogu głównym.
Jeśli miałaś problem ze ścieżkami to wystarczy wpisywać je względem tego pliku tj. np. ./images/logo.csv
| <link rel="stylesheet" href="style/style.css"> | ||
| <link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Montserrat:wght@400;700&display=swap"> | ||
| </head> | ||
| <body> |
There was a problem hiding this comment.
Warto ustawić margin: 0; padding: 0 dla body - wtedy nie ma takiej dziwnej ramki po bokach.
| <h1> FEATURES </h1> | ||
| <div class="container2"> | ||
| <div class="feature"> | ||
| <img src="responsive.svg" alt="Icon 1" class="icon"> |
There was a problem hiding this comment.
Ustawiłbym kolor bliższy białemu niż niebieskiemu :)
| <div class="container2"> | ||
| <div class="feature"> |
There was a problem hiding this comment.
Zwróć uwagę, że mamy tutaj "listę cech" dlatego zamiast div > div zrobiłbym ul > li - będzie to czytelniejsze
| <section class="white-section"> | ||
| <div class="col2"> |
There was a problem hiding this comment.
Uwaga na formatowanie kodu - trzymaj odpowiednie odstępy od lewej krawędzi
|
|
||
| <section class="grey-section"> | ||
| <header> | ||
| <div> |
There was a problem hiding this comment.
Ten div raczej jest zbędny :)
| <section class="pricing"> | ||
| <div class="pricing-plan"> |
There was a problem hiding this comment.
Uwaga na formatowanie.
PS. Ponieważ mamy "listę cen" to tutaj również można zastosować ul > li (każdy pakiet cenowy) zamiast. section > div
| </section> | ||
| <section class="white-section2"> | ||
| <header> | ||
| <div> |
| <section class="blue-section"> | ||
| <header class="download-free">Download FREE!</header> | ||
| <p>Take it for a ride for any of your |
| transform: scale(0.65); | ||
| /*margin-left: 2px /* spytac czemu tutaj nie da sie tego bardziej dosnac do lewej*/ |
There was a problem hiding this comment.
Trzeba nadać szerokość np. 100% i wtedy nie ma problemu - grafika się dostosuje do wielkosći ekranu. O tym będzie w następnym materiale :)
2 participants
hello @devmentor-pl please review the code