Conversation
|
🍏 Пройден линтинг и базовые тесты |
savichev-igor
left a comment
savichev-igor
left a comment
There was a problem hiding this comment.
Переключалка неинтересная
А тут картинка вообще болеет
Выглядит очень раздражающе, когда одинаковый контент прыгает
| @@ -0,0 +1 @@ | |||
| <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="13" height="13" viewBox="0 0 13 13"><path fill="#FAB600" d="M6.491 0l1.509 5h5l-4 3 2 5-4.5-3-4.5 3 2-5-4-3h5z"/></svg> No newline at end of file | |||
There was a problem hiding this comment.
Этот svg нигде не используется. Неиспользуемых файлов быть не должно!
| .item-category | ||
| { | ||
| display: inline-block; | ||
| color: #666; |
|
|
||
| .star-label | ||
| { | ||
| float: right; |
| <div class="card-top"> | ||
| <img class="item-pic" src="img/1.jpg" alt="1"> | ||
| </div> | ||
| <div class="card-middle"> |
There was a problem hiding this comment.
Слишком много div'ов, избавляйся от них
|
🚀 |
| @@ -0,0 +1 @@ | |||
| <svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="13" height="13" viewBox="0 0 13 13"><path fill="#FAB600" d="M6.491 0l1.509 5h5l-4 3 2 5-4.5-3-4.5 3 2-5-4-3h5z"/></svg> No newline at end of file | |||
There was a problem hiding this comment.
Этот svg нигде не используется. Неиспользуемых файлов быть не должно!
|
|
||
| <section class="item-card"> | ||
| <div class="card-top"> | ||
| <img class="item-pic" src="img/1.jpg" alt="1"> |
There was a problem hiding this comment.
"1" - плохой альтернативный текст для картинки
| </fieldset> | ||
| <div class="item-price"> | ||
| <h2 class="current-price">£750</h2> | ||
| <h3 class="old-price">£800</h3> |
There was a problem hiding this comment.
Какие же это заголовки?
Тут ты используешь h2/h3 для получения крупного текста. Так делать не нужно - заголовки только для заголовков!
| <li class="item-category"><a href="#">British Shorthair</a></li> | ||
| <li class="item-category"><a href="#">Chelmsford</a></li> | ||
| </ul> | ||
| <fieldset class="item-stars"> |
There was a problem hiding this comment.
Кажется ты спутал figure с fieldset :)
http://htmlbook.ru/html/fieldset
http://htmlbook.ru/html/figure
fieldset имеет смысл только внутри форм.
| </div> | ||
| </div> | ||
| <div class="card-bottom"> | ||
| <p class="item-description"> |
There was a problem hiding this comment.
item-description нет в css. Неиспользуемых классов быть не должно, как и лишних оберток.
2 элемента:
<div class="card-bottom"><p class="item-description">...
тоже самое одним:
<p class="card-bottom">...
| <img class="item-pic" src="img/1.jpg" alt="1"> | ||
| </div> | ||
| <div class="card-middle"> | ||
| <div class="item-info"> |
There was a problem hiding this comment.
item-info нет в css. Неиспользуемых классов быть не должно.
| { | ||
| display: inline-block; | ||
| vertical-align: top; | ||
| height: 100%; |
There was a problem hiding this comment.
У родителя (.item-card) не задана высота, соответственно height: 100% здесь никаким образом на верстку не повлияет.
|
🍅 |
4 participants
Посмотреть решение