Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Авинас Мария #44

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
54 changes: 30 additions & 24 deletions index.css
Original file line number Diff line number Diff line change
@@ -1,92 +1,98 @@
.card-to-list:checked ~ main .catPhoto
{
.list-view-checkbox:checked ~ main .cat-photo
{
width: 25%;
display: inline-block;
}

.card-to-list:checked ~ main .information
{
.list-view-checkbox:checked ~ main .information
{
display: inline-block;
vertical-align: top;
padding-left: 10px;
}

.card-to-list:checked ~ main .information .name
{
.list-view-checkbox:checked ~ main .information .name
{
width: 100%;
Copy link

@trixartem trixartem Nov 14, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Точно такое же свойство ниже
  2. Для чего ширину менять?

}

.card-to-list:checked ~ main .card
{
.list-view-checkbox:checked ~ main .card
{
width: 100%;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь тоже не понятно зачем устанавливать ширину 100%

}

header
{
.name-of-market

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

market-name, потому что of не несет смысловой нагрузки и лучше записать короче

{
font-size: 2em;
text-align: center;
background-color: white;
}

header:first-letter
{
.name-of-market header:first-letter
{
color: red;
}

.card
{
{
margin: 5px 0;
display: inline-table;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему inline-table?

width: 250px;
text-overflow: ellipsed;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не правильное свойство. Оно не будет работать.

}

.name a:hover
{
{
color: red;
}

.category:hover
{
{
color: gray;
text-decoration: underline;
}

.cat-photo:hover
{
border: 1px solid red;
}

.card:hover
{
{
background-color: lightgray;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше не описывать цвет словом, а с помощью hex записи

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше использовать hex запись, что бы запись была короче и этот цвет проще менять(делать светлее темнее и т. д.).

border: 1px solid black;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#000 короче

}

img
{
.cat-photo
{
max-width: 100%;
}

body
{
{
background-color: #ff6;
}

.name
{
{
width: 250px;
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
}

a
{
.name a
{
color: black;
}

.price .current
.price .current-price
{
color: red;
font-size: larger;
}

.image-for-list
.list-icon
{
width: 30px;
}
75 changes: 37 additions & 38 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,15 @@
<title>Мяндекс.Мурркет</title>
</head>
<body>
<header>
<header class="name-of-market">
Мяндекс.Муррркет<br>
<span>Магазин добрых котиков</span>
</header>
<br>
<input type="checkbox" class="card-to-list">
<img alt="list" src="spisok.png" title="Список" class="image-for-list">
<input type="checkbox" class="list-view-checkbox">
<img alt="иконка для изображения списка" src="spisok.png" title="Список" class="list-icon">
<main>
<div class="card">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

article

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Где семантика?

<img class="catPhoto" alt="котик №1" src="cats_photos/cat1.jpg">
<img class="cat-photo" alt="фото полосатого кота" src="cats_photos/cat1.jpg">
<div class="information">
<div class="name" >
<a href="#">Мурзик</a>
Expand All @@ -29,14 +28,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">12 мурлей</ins>
<del class="old">14 мурлей</del>
<ins class="current-price">12 мурлей</ins>
<del class="old-price">14 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №2" src="cats_photos/cat2.jpg">
<img class="cat-photo" alt="фото рыжего вислоухого кота" src="cats_photos/cat2.jpg">
<div class="information">
<div class="name">
<a href="#">Рыжик</a>
Expand All @@ -50,14 +49,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<del class="old">13 мурлей</del>
<ins class="current-price">11 мурлей</ins>
<del class="old-price">13 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №3" src="cats_photos/cat3.jpg">
<img class="cat-photo" alt="фото серого вислоухого кота" src="cats_photos/cat3.jpg">
<div class="information">
<div class="name">
<a href="#">Пушок</a>
Expand All @@ -71,13 +70,13 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<ins class="current-price">11 мурлей</ins>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №4" src="cats_photos/cat4.jpg">
<img class="cat-photo" alt="фото спящего кота" src="cats_photos/cat4.jpg">
<div class="information">
<div class="name">
<a href="#">Кеша</a>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лишняя вложенность, кажется что достаточно одной ссылки

Expand All @@ -91,14 +90,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">10 мурлей</ins>
<del class="old">12 мурлей</del>
<ins class="current-price">10 мурлей</ins>
<del class="old-price">12 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №5" src="cats_photos/cat5.jpg">
<img class="cat-photo" alt="фото рыжего полосатого кота" src="cats_photos/cat5.jpg">
<div class="information">
<div class="name">
<a href="#">Соня</a>
Expand All @@ -112,13 +111,13 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">12 мурлей</ins>
<ins class="current-price">12 мурлей</ins>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №6" src="cats_photos/cat6.jpg">
<img class="cat-photo" alt="фото котенка в корзинке" src="cats_photos/cat6.jpg">
<div class="information">
<div class="name">
<a href="#">Великий и ужасный Бонифаций IV, но можно просто Боня</a>
Expand All @@ -132,13 +131,13 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">13 мурлей</ins>
<ins class="current-price">13 мурлей</ins>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №7" src="cats_photos/cat7.jpg">
<img class="cat-photo" alt="фото рыжего пушистого котёнка" src="cats_photos/cat7.jpg">
<div class="information">
<div class="name">
<a href="#">Кнопа</a>
Expand All @@ -152,13 +151,13 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">13 мурлей</ins>
<ins class="current-price">13 мурлей</ins>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №8" src="cats_photos/cat8.jpg">
<img class="cat-photo" alt="фото рыжего пушистого кота" src="cats_photos/cat8.jpg">
<div class="information">
<div class="name">
<a href="#">Стеша</a>
Expand All @@ -172,14 +171,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<del class="old">13 мурлей</del>
<ins class="current-price">11 мурлей</ins>
<del class="old-price">13 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №9" src="cats_photos/cat9.jpg">
<img class="cat-photo" alt="фото спящего полостаого кота" src="cats_photos/cat9.jpg">
<div class="information">
<div class="name">
<a href="#">Барсик</a>
Expand All @@ -193,14 +192,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<del class="old">13 мурлей</del>
<ins class="current-price">11 мурлей</ins>
<del class="old-price">13 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №10" src="cats_photos/cat10.jpg">
<img class="cat-photo" alt="фото рыжего пушистого кота на снегу" src="cats_photos/cat10.jpg">
<div class="information">
<div class="name">
<a href="#">Муся</a>
Expand All @@ -214,14 +213,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">10 мурлей</ins>
<del class="old">12 мурлей</del>
<ins class="current-price">10 мурлей</ins>
<del class="old-price">12 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №11" src="cats_photos/cat11.jpg">
<img class="cat-photo" alt="фото полостаго кота" src="cats_photos/cat11.jpg">
<div class="information">
<div class="name">
<a href="#">Васька</a>
Expand All @@ -235,14 +234,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<del class="old">13 мурлей</del>
<ins class="current-price">11 мурлей</ins>
<del class="old-price">13 мурлей</del>
</div>
<div class="description">Самый лучший котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №12" src="cats_photos/cat12.jpg">
<img class="cat-photo" alt="фото чёрного кота" src="cats_photos/cat12.jpg">
<div class="information">
<div class="name">
<a href="#">Ужас, летящий на крыльях ночи</a>
Expand All @@ -256,14 +255,14 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">11 мурлей</ins>
<del class="old">13 мурлей</del>
<ins class="current-price">11 мурлей</ins>
<del class="old-price">13 мурлей</del>
</div>
<div class="description">Самый добрый котик!</div>
</div>
</div>
<div class="card">
<img class="catPhoto" alt="котик №13" src="cats_photos/cat13.jpg">
<img class="cat-photo" alt="фото серого кота с пятном на морде в виде усов" src="cats_photos/cat13.jpg">
<div class="information">
<div class="name">
<a href="#">Мистер Усач</a>
Expand All @@ -277,7 +276,7 @@
<span>&#9734;</span>
</div>
<div class="price">
<ins class="current">14 мурлей</ins>
<ins class="current-price">14 мурлей</ins>
</div>
<div class="description">Самый оригинальный котик!</div>
</div>
Expand Down