D
D
Denis Lipatov2016-09-26 16:32:53
css
Denis Lipatov, 2016-09-26 16:32:53

Tell me what is wrong with the layout, where are the errors?

I want criticism, direct me to mistakes, otherwise I don’t catch up with something, what’s wrong? Here is the layout of the test, tell me what is wrong in the code?
https://yadi.sk/d/SHGaGXIlvjfmr

Answer the question

In order to leave comments, you need to log in

2 answer(s)
Z
Zam_Bo, 2016-09-26
@Zam_Bo

Well, I, for one, do not like the big cascade that you meet in places.

R
Raz-Dva, 2016-09-26
@Raz-Dva

Of course, I'm not an expert (I've been working as a layout designer for about 5 months), but I'll tell you about those mistakes that I myself came across.
- if you add a phone number. It's better to make div.tel>p (or span) and then check by adding or removing phone numbers
- in the header, I would do it not through col, but simply divs in one line and align it through vertical-align:midlle. In such a case, there is no need for margins at the top and the blocks would always be vertically aligned in the middle.
- I do sprites in one file. No need to break for social networks, for stars. And try to make them in one file.
Well, you should always check for the amount of text (if there is a lot of text or little), also with images (if they insert a large or smaller image from the admin panel) whether the layout will work. The price may also change. It's like my opinion. I think that the more experienced will find something else, just not everyone wants to waste their time.

Didn't find what you were looking for?

Ask your question

Ask a Question

731 491 924 answers to any question