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

margin for grid #1384

Closed
wants to merge 3 commits into from
Closed

margin for grid #1384

wants to merge 3 commits into from

Conversation

DissNik
Copy link
Member

@DissNik DissNik commented Dec 17, 2024

issue #1379

@shevelev-anatoliy
Copy link
Contributor

shevelev-anatoliy commented Dec 18, 2024

image

image
image

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

Я не совсем понимаю, почему здесь применяется отступ и сверху и снизу? Может оставить отступ только сверху? Или это ломает верстку где-то в других местах?

.form-group+.form-group {
    margin-bottom: 1.25rem;
    margin-top: 1.25rem;
}

@DissNik
Copy link
Member Author

DissNik commented Dec 18, 2024

а между ними нет LineBreak?

@shevelev-anatoliy
Copy link
Contributor

а между ними нет LineBreak?

Не, именно маргины суммируются, т.к. там грид, то не происходит схлапывания маргинов.

@shevelev-anatoliy
Copy link
Contributor

shevelev-anatoliy commented Dec 18, 2024

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

@DissNik
Copy link
Member Author

DissNik commented Dec 18, 2024

Внес коррективы, но может это вылезти где нибудь боком, я думаю нижний отступ нужен был как раз для grid collapse, может быть и еще для каких то компонентов

@DissNik
Copy link
Member Author

DissNik commented Dec 18, 2024

я пробверил Grid, Flex, Collapse, Tabs может что-то упустил

@DissNik
Copy link
Member Author

DissNik commented Dec 18, 2024

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

не зря )

есть два создания отступов: отступы задают родители или сами элементы, возможно стоит перейти на задание отступов через родителя

@DissNik
Copy link
Member Author

DissNik commented Dec 18, 2024

#1386 попробовал убрать отступы у элементов, можно посмотреть

@lee-to lee-to closed this Dec 18, 2024
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.

3 participants