1. responsive interface!
This is insanely important.
All interactively, should respond to user actions
the desired styles for the :hover pseudo-class for button
when you hover over links to social.networks, they also need to react. For example, the simplest opacity: .8
And don't forget about the cursor: pointer for just what you can interact
2. not seen the layout, but are You sure that the buttons have to be doubtful this grey border?
I think it is better (green arrow) border: 0
remove orange outline when :focus (blue arrow)
3. the phone needs to be a link and then it is clickable
<a href="tel:+74956660266">+7 (495) 666-02-66</a>
4. HOW TO FIND US? in the footer, as I understand it, this should also be a link
5. I would have removed the outline from the input in focus
and placeholder should disappear when the cursor is in the field, and it disappears from you only when you enter
You did the right padding-left from the input fields, but why doesn't the padding-right. In General, it's not very nice.
Doing the right thing by using the time tag)
In General, the green arrows signify that you have correctly used the tags h2
Reds say about the misuse of these tags
Where the red arrows I would suggest to use div
is simply the building blocks of semantics
headings and paragraphs have semantic meaning, so it is advised to use them where they're needed.
When I started, a mentor threw this message:
Paragraphs/Paragraphs level semantics must be used to markup content that should not be used in end-to-end blocks, since they are not content.
In fact, the paragraph/the paragraph is the same thing as in the Russian language, and should only be used to markup text.
6. the footer is not pushed to the bottom