Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

init #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

init #3

wants to merge 2 commits into from

Conversation

sameoldmadness
Copy link

No description provided.


h1 {
margin: 46px 0 104px;
padding-bottom: 7px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

padding: 0 0 7px 0; чуток короче, есть нет необходимости сохранить определённые заранее.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я правильно понимаю, мы пару байт экономим?

Читаемость, на мой взгляд, резко падает.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

да, экономим, и читаемость тут не падает, ты же свойство выше нормально читаешь?
margin: 46px 0 104px -- понятно что оно разложится в

margin-top: 46px;
margin-right: 0;
margin-bottom: 104px;
margin-left: 0;

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Строку выше я читаю как «Я хочу выставить отступ снизу и обнулить остальные отступы».

Как поддерживающему код разработчику понять, что я обнуляю отступы только для экономии места?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Даже так, правильнее экономнее будет padding: 0 0 7px;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants