Skip to content

MRK-5063: Компоненты: Новые стили чипсов в ките (evo-chip) (v6)#1282

Open
andreykiselev92 wants to merge 4 commits into6.xfrom
feature/MRK-5063_v6
Open

MRK-5063: Компоненты: Новые стили чипсов в ките (evo-chip) (v6)#1282
andreykiselev92 wants to merge 4 commits into6.xfrom
feature/MRK-5063_v6

Conversation

@andreykiselev92
Copy link
Copy Markdown
Collaborator

No description provided.

@harlamenko harlamenko changed the base branch from 6.x-backup to 6.x November 24, 2025 14:01
@harlamenko harlamenko deleted the branch 6.x November 24, 2025 14:38
@harlamenko harlamenko closed this Nov 24, 2025
@harlamenko harlamenko reopened this Nov 24, 2025
@harlamenko harlamenko force-pushed the 6.x branch 2 times, most recently from 6feedbf to 808d226 Compare November 24, 2025 14:48
display: flex;
align-items: center;
--evo-chip-background-color: #{mixins.$color-background-grey};
--evo-chip-background-color-hover: #{mix(mixins.$color-white, mixins.$color-background-grey, 20%)};
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Давай не будем этой функцией здесь пользоваться. Во-первых, она даёт немного мусорные результаты, а во-вторых, у нас есть стандартные цвета, на которые ориентируются другие компоненты:

$color-grey-hover
$color-grey-active

$root: &;

border-radius: math.div($height, 2);
--evo-chip-height: 32px;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Внутренние переменные, до которых не достучаться извне, лучше называть с другим префиксом.

Я обычно называю так: --inner-chip-height

}
&:hover,
#{$root}__input:focus ~ & {
background-color: var(--evo-chip-background-color-hover);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Чтобы не делать такие сложные конструкции с селекторами, которые сильно зависят от темплейта, заведи внутренние переменные:

--inner-background-color
--inner-text-color
--inner-cursor

и тд, примени их к нужным свойствам блоков и меняй их значение так, как тебе нужно.

}

input {
&_size-large {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Опиши ещё размер normal явно, пожалуйста. Так будет гораздо чётче.

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

image

Поэтому правый отступ можно было бы в отдельную переменную затолкать.

И можно также добавить БЭМ-модификаторы контейнеру, которые говорят о том, что у чипсы есть счётчик (has-counter) или крестик (is-closable). Так будет проще этот правый отступ применить.

</span>
</div>

<div *ngIf="hasCounter" class="chip__counter">{{ counter }}</div>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Давай используем здесь evo-counter компонент и уберём лишние стили.

semantic-release-bot and others added 2 commits February 3, 2026 07:00
## [6.22.2](v6.22.1...v6.22.2) (2026-02-03)

### Bug Fixes

* **icons:** add kuper logos ([35d99e1](35d99e1))
@andreykiselev92 andreykiselev92 force-pushed the feature/MRK-5063_v6 branch 4 times, most recently from 48a05b1 to 870492b Compare February 20, 2026 12:15
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.

4 participants