Skip to content

hotfix(slider): починить выравнивание label по правому и левому краю [DS-17047]#2219

Merged
dHIM24 merged 2 commits into
masterfrom
DS-17047
May 22, 2026
Merged

hotfix(slider): починить выравнивание label по правому и левому краю [DS-17047]#2219
dHIM24 merged 2 commits into
masterfrom
DS-17047

Conversation

@dHIM24
Copy link
Copy Markdown
Contributor

@dHIM24 dHIM24 commented May 21, 2026

  • Исправлен регресс выравниваний крайних подписей (min/max) под слайдером.

Чек лист

  • Задача сформулирована и описана в JIRA
  • В названии ветки есть айдишник задачи в JIRA (fix/DS-1234), ссылку прикреплять не надо
  • У реквеста осмысленное название feat(...) или fix(...) по conventional commits (https://www.conventionalcommits.org)
  • Код покрыт тестами и протестирован в различных браузерах
  • Добавленные пропсы добавлены в демки и описаны в документации
  • К реквесту добавлен changeset

Если есть визуальные изменения

  • Прикреплено изображение было/стало

Slider

Снимок экрана 2026-05-21 в 08 41 53

SliderInput [others-themes]

Снимок экрана 2026-05-21 в 08 42 09

SliderInput [site-theme]

Снимок экрана 2026-05-21 в 08 42 05

@dHIM24 dHIM24 requested a review from Oladii May 21, 2026 05:44
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 21, 2026

🦋 Changeset detected

Latest commit: ea3e632

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

This PR includes changesets to release 4 packages
Name Type
@alfalab/core-components-themes Minor
@alfalab/core-components-slider Patch
@alfalab/core-components-slider-input Patch
@alfalab/core-components 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

@dHIM24 dHIM24 changed the title hotfix(slider): починить выравнивание label по правому и левому краю hotfix(slider): починить выравнивание label по правому и левому краю [DS-17047] May 21, 2026
@coveralls
Copy link
Copy Markdown

coveralls commented May 21, 2026

Coverage Report for CI Build 26284586951

Coverage remained the same at 82.241%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 11612
Covered Lines: 9622
Line Coverage: 82.86%
Relevant Branches: 2206
Covered Branches: 1742
Branch Coverage: 78.97%
Branches in Coverage %: Yes
Coverage Strength: 234.66 hits per line

💛 - Coveralls

@github-actions
Copy link
Copy Markdown
Contributor

Demo build (default)

https://core-ds.github.io/core-components/2219

@github-actions
Copy link
Copy Markdown
Contributor

Demo build (alfasans)

https://core-ds.github.io/core-components/2219-alfasans

}

.component .slider {
--slider-pips-offset-inline: 0;
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.

все переменные должны быть в :root

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Comment thread packages/slider/src/vars.css Outdated
--slider-clickable-area-half-size: calc(var(--slider-clickable-area-size) / 2);
--slider-origin-width: calc(100% - 16px);
--slider-origin-right: 8px;
--slider-pips-offset-top: var(--gap-6);
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.

для чего переменные заведены?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

спасибо

Comment thread .changeset/tired-kings-chew.md Outdated
---
'@alfalab/core-components-slider': patch
'@alfalab/core-components-slider-input': patch
'@alfalab/core-components': patch
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.

либо исключить этот пункт, либо дописать ##### Slider, SliderInput ниже

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@Oladii
Copy link
Copy Markdown
Contributor

Oladii commented May 21, 2026

сломался дефолт:
В мастере так
image

В ветке точка сдвинулась
image

@dHIM24 dHIM24 force-pushed the DS-17047 branch 2 times, most recently from 2dd5353 to 7d5586c Compare May 22, 2026 05:55
@dHIM24 dHIM24 closed this May 22, 2026
@dHIM24 dHIM24 reopened this May 22, 2026
@dHIM24 dHIM24 reopened this May 22, 2026
@dHIM24 dHIM24 closed this May 22, 2026
@dHIM24 dHIM24 reopened this May 22, 2026
@dHIM24 dHIM24 merged commit 848edee into master May 22, 2026
31 checks passed
@dHIM24 dHIM24 deleted the DS-17047 branch May 22, 2026 14:04
@core-ds-app core-ds-app Bot mentioned this pull request May 22, 2026
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.

5 participants