bug(UIKIT-1017,ui,CheckboxField): Выравнивание лейбла по вертикали относительно чекбокса

[Kostayne]

…тикали

Чеклист

• Написать тесты для реализованного функционала
• Описать jsdoc для экспортируемых функций, компонентов
• Описать jsdoc для props экспортируемых компонентов
• Реализовать или обновить storybook для компонента
• Проверить код на соответствие style guide
• Запросить ревью у дизайнера, если был изменен ui или создан новый компонент

[mfrolov89]

bug(UIKIT-1017,ui,CheckboxField)
https://github.com/kaluga-astral/frontend/blob/main/CONTRIBUTING.md

[Kostayne]

bug(UIKIT-1017,ui,CheckboxField) https://github.com/kaluga-astral/frontend/blob/main/CONTRIBUTING.md

CheckboxField находится в components, так что ui там быть не должно, а вот description коммита стоит поменять

[mfrolov89]

bug(UIKIT-1017,ui,CheckboxField) https://github.com/kaluga-astral/frontend/blob/main/CONTRIBUTING.md

CheckboxField находится в components, так что ui там быть не должно, а вот description коммита стоит поменять

Посмотри доку, там определен определенный перечень scope. Components там нет, есть ui, который как раз и обозначает сomponents, сопутствующие примеры там есть. О исторических причинах такого нейминга наверняка не подскажу, но скорее всего дело в краткости и простоте написания. При это учитывай что длина названия тоже ограничена. Если считаешь что нужно вводить другой нейминг, то давай отдельно обсуждать.

[Kostayne]

Посмотри доку, там определен определенный перечень scope. Components там нет, есть ui, который как раз и обозначает сomponents, сопутствующие примеры там есть. О исторических причинах такого нейминга наверняка не подскажу, но скорее всего дело в краткости и простоте написания. При это учитывай что длина названия тоже ограничена. Если считаешь что нужно вводить другой нейминг, то давай отдельно обсуждать.

Окей, запомню

[Kostayne]

Наткнулся на сообщение от husky:
"✖ scope must be one of [components, features, form, icons, ui, server, UIKIT-XXXX] [function-rules/scope-enum]".

Если скоупы перечисленные в нем неактуальны, то нужно обновить @astral/commitlint-config

[mfrolov89]

Задача была на компонент Checkbox. У CheckboxField есть такая же проблема, которую ты исправляешь, на запроса на это не было

[Kostayne]

Задача была на компонент Checkbox. У CheckboxField есть такая же проблема, которую ты исправляешь, на запроса на это не было

Сам по себе чекбокс не содержит лейбл, это видно по коду самого чекбокса и коду в сторибуке. Там для подобных целей используется FormControlLabel, его менять определенно точно не стоит, тк вместо чекбокса он может содержать RadioButton, этот случай мы затрагивать не должны.

Именно поэтому и был затронут CheckboxField

[github-actions]

🎉 This PR is included in version 3.102.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀