Черных Артём #64
Черных Артём #64
Conversation
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
There was a problem hiding this comment.
Сделано как-то некрасиво..
-
Грустные переключалки какие-то: серые инпуты.
-
Картинки не прогрузились.
-
hoverна цену вызывает желание закрыть вкладку).. -
hoverна фотку как-то странно вылазит из обводки.
-
Тут бы явно не помешал
padding.
-
Списком опять же странно выглядит.
| </head> | ||
| <body> | ||
| <input type="radio" name="view" value="grid" checked>Плитка | ||
| <input type="radio" name="view" value="list" class="list">Список | ||
| <br> |
| .cat img:hover | ||
| { | ||
| bottom: -5%; | ||
| left: -5%; |
|
|
||
| .price | ||
| { | ||
| color: #b00000; |
| .price:hover::after | ||
| { | ||
| color: #000; | ||
| content: 'купи меня'; |
There was a problem hiding this comment.
Весь контент лучше размещать в HTML, ибо получишь страдания при локализации сайта..
|
🚀 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 |
|
Верстка далековата от макета, сделайте, пожалуйста как можно ближе к макету. И не все задания выполнены. 🍅 |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 Приблизил к макету. |
|
Замечания по верстке:
|
|
|
||
| .list:checked ~ .note:hover | ||
| { | ||
| transform: scaleX(1); |
There was a problem hiding this comment.
Для чего переопределять значение? Можно же добавлять эффект, если выбран режим плиток
| .price | ||
| { | ||
| color: #b00; | ||
| font: bold 1.5em sans-serif; |
There was a problem hiding this comment.
Это даёт какие-нибудь преимущества?
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 Изменил |
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 |
Посмотреть решение