A
A
artem16062021-11-14 23:20:44
code review
artem1606, 2021-11-14 23:20:44

Please rate the quality of the layout?

Please rate the quality of the layout. Let me know what can be changed or fixed.
https://bakum1981.github.io/agro-world/

Answer the question

In order to leave comments, you need to log in

2 answer(s)
N
Nikolay Savelyev, 2021-11-15
@AgentSmith

This is just a regular template from Bootstrap.
You don't need to change or fix anything. I'm ok.
The video is completely off topic. what does the seeds and the sea have to do with Thailand?

A
Alexey Dubrovin, 2021-11-15
@alekcena

1) The main tag is not supported by most.
2) You have constructs with different sequences in your code. What is not good.

.любой класс{
width:
display:
}
// Потом другой класс
.любой класс{
display:
width:
}

3) Also, if you use BEM, then use it completely and not partially.
Constructs
like .input._error is generally what.
What for after a point to put still the special underscore character.
4)HTML is a markup language that uses semantics.
For text, you need to use the appropriate tags.
p span, not div
5) You are using headings. h1 and h3. Where did h2 go? If it is not there, why h3
6) When you make a menu or any elements that need to be clicked on. Don't make the click area the same size as text. Always try to make it a little bigger. Especially critical on mobile phones.
If you choose to accidentally hurt, or poke 10 times until you hit. That's better 1
The animations are not bad. There are no lags. Or serious drawdowns
+- Normal adaptive.
You can still work on the accessibility of the content. I am attaching a link to what and why.

Didn't find what you were looking for?

Ask your question

Ask a Question

731 491 924 answers to any question