Skip to content

Меняем внутренние отступы у тегов и фильтртегов во всех темах#1416

Merged
fulcanellee merged 3 commits intomasterfrom
feat/filter-tag_themezation
Oct 24, 2024
Merged

Меняем внутренние отступы у тегов и фильтртегов во всех темах#1416
fulcanellee merged 3 commits intomasterfrom
feat/filter-tag_themezation

Conversation

@fulcanellee
Copy link
Contributor

https://jira.moscow.alfaintra.net/browse/DS-7037

Изменены отступы в tag и filter-tag

@changeset-bot
Copy link

changeset-bot bot commented Oct 15, 2024

🦋 Changeset detected

Latest commit: 462bf2c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@alfalab/core-components-filter-tag Minor
@alfalab/core-components-tag Minor
@alfalab/core-components-vars Minor
@alfalab/core-components-pagination Patch
@alfalab/core-components-select-with-tags Patch
@alfalab/core-components-tabs Patch
@alfalab/core-components-table Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@fulcanellee fulcanellee force-pushed the feat/filter-tag_themezation branch from a81c6c4 to 35786e3 Compare October 15, 2024 08:33
@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@fulcanellee fulcanellee requested a review from Oladii October 15, 2024 11:52
@Oladii Oladii changed the title Feat/filter tag themezation Меняем внутренние отступы у тегов и фильтртегов во всех темах Oct 17, 2024
@dedanu dedanu self-requested a review October 17, 2024 10:51
@ouchnomad ouchnomad self-requested a review October 17, 2024 13:38
Copy link

@YuriKiyan YuriKiyan left a comment

Choose a reason for hiding this comment

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

От сайта OK

Copy link
Contributor

Choose a reason for hiding this comment

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

думаю стоит 28 захардкодить, так как это не совсем легальный гэп чтобы выносить его в переменные. В макете используется конструкция из суммы 2х гэпов 24 и 4 которые относятся к разным деталям тэга, но в коде исторически это один отступ.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

поправил

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@reme3d2y
Copy link
Contributor

@Oladii тут меняются размеры компонента, потенциально это может поломать где-нибудь верстку. Мы считаем это миноркой?

@fulcanellee fulcanellee merged commit 402c3e2 into master Oct 24, 2024
@fulcanellee fulcanellee deleted the feat/filter-tag_themezation branch October 24, 2024 10:47
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.