19/20 и 20/20

Курсы §18. Селекторы. Часть 2
1

Добрый вечер! По традиции после прохождения урока, выложу код для критики. Хочется узнать явные ошибки. Вроде все было просто, но как обычно бывает, возможно, что-то можно было написать короче.

19/20 CSS:

html,
body {
margin: 0;
padding: 0;
font-family: "Arial", sans-serif;
line-height: 30px;
}

body {
min-width: 550px;
min-height: 290px;
padding: 10px;
}

.two-cards {
width: 350px;
height: 250px;
padding: 10px;
text-align: center;
}

.two-cards div {
position: relative;
display: inline-block;
width: 120px;
height: 180px;
margin: 0 10px;
margin-top: 30px;
vertical-align: middle;
background-color: white;
border-radius: 5px;
box-shadow: 0px 2px 4px rgba(0, 0, 0, 0.8);
}

.ace-heart {
background: url("/assets/course57/heart.svg") no-repeat 50% 50%;
background-size: 50px auto;
}

.two-spade {
background: url("/assets/course57/spade.svg") no-repeat 50% 50%;
background-size: 50px auto;
}

/* Ширина иконок, не изменяйте */
.two-cards div::before,
.two-cards div::after {
position: absolute;
width: 20px;
height: 50px;
font-size: 25px;
font-weight: bold;
}

div.ace-heart::before,
div.ace-heart::after {
color: red;
content: "A";
background: url("/assets/course57/heart.svg") 50% 100% no-repeat;
background-size: 25px auto;
}

div.two-spade::before,
div.two-spade::after {
color: black;
content: "2";
background: url("/assets/course57/spade.svg") 50% 100% no-repeat;
background-size: 25px auto;
}

.two-cards div::before {
top: 5px;
left: 5px;
}
/* Поворот нижней иконки */
.two-cards div::after {
transform: rotate(180deg);
bottom: 5px;
right: 5px;
}

20/20 CSS:

html,
body {
margin: 0;
padding: 0;
font-family: "Arial", sans-serif;
}

body {
min-width: 550px;
min-height: 250px;
padding: 10px;
}

.cards {
position: relative;
width: 380px;
text-align: left;
}

.cards li {
width: 37px;
height: 55px;
}

.cards li::before {
font-family: "Courier", monospace;
}

li:not(:first-child) {
margin-left: -20px;
}

.cards::after {
content: "";
position: absolute;
right: 0;
top: 0;
display: block;
width: 37px;
height: 55px;
border: 1px dashed #33bb33;
border-radius: 5px;
}

li:nth-child(4n + 1) {
background-color: #ffcc33;
}

ul:not(:first-of-type) li:nth-child(4n + 2) {
background-color: #0099ff;
}

ul:nth-of-type(2n) li:nth-child(4n) {
background-color: #ff3300;
}

ul:nth-of-type(3) li.heart:not(.ten):not(.five) {
background-color: #339933;
}

ul:first-of-type li:nth-child(2n) {
background-color: #996666;
2

Чтобы укоротить код, я немного схитрил : (не стал указывать вначале .cards, чтобы сделать приоритетность ниже)
li:nth-of-type(4n+2) {
background-color: #0099ff;
}

Тут решил обойтись без 2ойного not :
.cards:nth-of-type(3) li:nth-child(3n+8) {
background-color: #339933;
}

В остальном как у вас, за исключением того, что использую класс .cards вместо тега ul .

1 лайк
3

Соглашусь, ваш вариант лучше.