Ваш код работает хорошо, как кажется. единственное, что я рекомендую использовать, это repeat(multiples, value)
, если все значения равны, а не записывать необходимые значения каждого элемента сетки.
Также при изменении чего-либо в точке останова пишите только необходимое изменение без дублирования других свойства, потому что они все еще будут применяться.
Следующий пример похож на ваш, но с минимальными значениями, запустите фрагмент, используя полную страницу, и измените размер страницы, чтобы заметить разницу.
.product {
background-color: cyan;
}
.grid-3-2 {
display: grid;
grid-template-columns: repeat(3, 100px);
grid-template-rows: repeat(2, 50px);
grid-row-gap: 10px;
grid-column-gap: 20px;
}
@media(max-width: 1200px) {
.grid-3-2 {
grid-template-columns: repeat(2, 100px);
grid-template-rows: repeat(3, 50px);
}
}
<ol class="grid-3-2">
<li class="product"></li>
<li class="product"></li>
<li class="product"></li>
<li class="product"></li>
<li class="product"></li>
<li class="product"></li>
</ol>