19/20 100% Постарался оптимизировать

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

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);
}

/* Ширина иконок, не изменяйте */
.two-cards div::before,
.two-cards div::after {
width: 20px;
}

/* Поворот нижней иконки */
.two-cards div::after {
transform: rotate(180deg)!important;
}

/Свой код/

.ace-heart,
.ace-heart::before,
.ace-heart::after {
color: red;
background-image: url("/assets/course57/heart.svg");
}

.two-spade,
.two-spade::before,
.two-spade::after {
color: black;
background-image: url("/assets/course57/spade.svg");
}

.ace-heart,
.two-spade {
position: relative;
background-repeat: no-repeat;
background-position: 50% 50%;
background-size: 50px auto;
font-size: 25px;
font-weight: bold;
}

.ace-heart::after,
.ace-heart::before,
.two-spade::before,
.two-spade::after {
position: absolute;
padding-bottom: 20px;
background-repeat: no-repeat;
background-position: 50% 100%;
background-size: 25px auto;
}

.ace-heart::before,
.two-spade::before {
top: 5px;
left: 5px;
}

.ace-heart::before,
.ace-heart::after {
content: “A”;
}

.ace-heart::after,
.two-spade::after {
bottom: 5px;
right: 5px;
}

.two-spade::before,
.two-spade::after {
content: “2”;

2
  1. Если мы решили оптимизировать наш код, то почему свойство background не писать одной строчкой?
    (background: url("/assets/course57/heart.svg") no-repeat 50% 100%)
  2. Я бы все таки сильно не увлекался “оптимизацией”, где мы выносим по 1-2 общих свойства - код разрастется так, что мозоли от скролла появяться при попытке найти нужное свойство. Например, “content” оставил вместе с другими.
3

Вот такой вариант
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);

}
.two-cards .ace-heart{
background: white url("/assets/course57/heart.svg") no-repeat 50% 50%;
background-size:50px auto;
}
.two-cards .two-spade{
background: white url("/assets/course57/spade.svg") no-repeat 50% 50%;
background-size:50px auto;
}
.two-cards .ace-heart::before,
.two-cards .ace-heart::after{
content:“A”;
font-weight:bold;
color: red;
font-size:25px;
padding-bottom:20px;
position:absolute;
}
.two-cards .ace-heart::before{
left:5px;
top:5px;
background:url(/assets/course57/heart.svg) no-repeat 50%100%;
background-size:25px auto;
}
.two-cards .ace-heart::after{
right:5px;
bottom:5px;
background:url(/assets/course57/heart.svg) no-repeat 50%100%;
background-size:25px auto;
}
.two-cards .two-spade::before,
.two-cards .two-spade::after{
content:“2”;
font-weight:bold;
color: black;
font-size:25px;
padding-bottom: 20px;
position:absolute;
}
.two-cards .two-spade::before{
left:5px;
top:5px;
background:url(/assets/course57/spade.svg) no-repeat 50%100%;
background-size:25px auto;
}
.two-cards .two-spade::after{
right:5px;
bottom:5px;
background:url(/assets/course57/spade.svg) no-repeat 50%100%;
background-size:25px auto;
}

/* Ширина иконок, не изменяйте */
.two-cards div::before,
.two-cards div::after {
width: 20px;
}

/* Поворот нижней иконки */
.two-cards div::after {
transform: rotate(180deg);