Антон Усачев

$('.block1 .block2')...

Но лучше дать единый класс для всех элементов, к которым идет обращение, если предполагается делать с ними одно и то же действие.

button {
    outline: none !important; 
}

input[type="button"]::-moz-focus-inner {
    border: 0;
}

А для фокуса нужно придумать свой стиль для кнопки, чтобы было понятно, что она выделена по табу, например:

button:focus {
	box-shadow: 0 0 0 2px blue;
}

Слишком много ненужного кода. Особенно, когда можно обойтись парой строчек цсс:

.main-color:hover .item img {
	transition: opacity .3s ease;
}

.main-color:hover .item img.item__img-color {
	opacity: 0;
}

/* item, который должен оставаться цветным получает класс active */
.main-color:hover .item img.item__img-color:hover,
.main-color:hover .item.active img.item__img-color {
	opacity: 1;
}

В спецификации HTML5 появился еще атрибут form, который позволяет выносить сабмит за пределы формы:

<form id="myform" method="get" action="something.php">
    <input type="text" name="name" />
</form>

<input type="submit" form="myform" />