Skip to content

PLASMA-6352: Slider centering was fixed #653

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.

Sign up for GitHub

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

Jump to bottom
Open
SerraMorec wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

PLASMA-6352: Slider centering was fixed #653

SerraMorec wants to merge 4 commits into from

Conversation

@SerraMorec
Copy link
Collaborator

@SerraMorec SerraMorec commented Jan 26, 2026 •
edited
Loading

sdds-uikit-compose

Slider

  • Исправлено центрирование полосы слайдера относительно label.

What/why changed

Реализован кастомный layout для расположения и выравнивания дочерних composable

Снимок экрана 2026-01-26 в 14 09 47 Снимок экрана 2026-01-26 в 14 32 08

@SerraMorec SerraMorec added wip Work In Progress compose labels Jan 26, 2026
@salutedevs
Copy link
Collaborator

salutedevs commented Jan 26, 2026 •
edited
Loading

📦 Собранные APK

📲 Как установить APK на Android-устройство:

  1. Распакуйте архив.
  2. Скопируйте нужный .apk на устройство (через USB, Telegram, Google Drive и т.д.).
  3. Откройте .apk на устройстве и подтвердите установку.
    Если потребуется, включите Разрешение на установку из неизвестных источников в настройках устройства.

💻 Установка через USB (ADB):

  1. Включите Отладку по USB на устройстве (Настройки → Для разработчиков).
  2. Подключите устройство к компьютеру по USB.
  3. Убедитесь, что устройство отображается: 
    adb devices
  4. Установите .apk: 
    adb install path/to/your.apk
    Если приложение уже установлено и нужно обновить: 
    adb install -r path/to/your.apk

@salutedevs
Copy link
Collaborator

salutedevs commented Jan 26, 2026 •
edited
Loading

📘Артефакты документации опубликованы:

@SerraMorec SerraMorec force-pushed the bugfix/slider-centering-compose branch from eed6bdf to ab37019 Compare January 29, 2026 05:30
@SerraMorec SerraMorec removed the wip Work In Progress label Jan 30, 2026
@SerraMorec SerraMorec force-pushed the bugfix/slider-centering-compose branch from dae1f4d to 3feb134 Compare January 30, 2026 08:08
@SerraMorec SerraMorec requested a review from luizasok February 2, 2026 07:23
raininforest
raininforest reviewed Feb 2, 2026
View reviewed changes
sdds-core/uikit-compose/src/main/kotlin/com/sdds/compose/uikit/Slider.kt Outdated
limMargin: Int,
labMargin: Int,
) : MeasurePolicy {
private val sizes: MeasuredSizes = MeasuredSizes().apply {
Copy link
Collaborator

@raininforest raininforest Feb 2, 2026

Choose a reason for hiding this comment

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

Этот класс с var внутри - очень сомнительное решение. Его очень легко мутировать и потом не понять, почему и в какой момент изменились его var свойства. Также сомнительна коллекция с Placeable. Почему нельзя обойтись без этих сущностей?

Copy link
Collaborator Author

@SerraMorec SerraMorec Feb 2, 2026

Choose a reason for hiding this comment

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

я больше не придумал как это все упорядочить более менее.
как ты можешь заметить MeasuredSizes мне нужны для вычисления каждого из последующих размеров, а так же для вычисления смещения, без них было бы большое количество аргументов в каждой из функции. я посчитал, что data class - будет не очень удобно, так как мне пришлось бы каждый раз копировать для записи новых размеров. ну а коллекция с Placeable - я не представляю какой бы был тогда layout без нее. я готов к обсуждению, если скажешь как это можно все сделать без дополнительных коллекций и гору логики припрятать)))

Copy link
Collaborator Author

@SerraMorec SerraMorec Feb 2, 2026

Choose a reason for hiding this comment

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

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

@SerraMorec SerraMorec force-pushed the bugfix/slider-centering-compose branch from 3feb134 to c88fa78 Compare February 2, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raininforest raininforest raininforest left review comments

@angirb angirb angirb approved these changes

@malilex malilex Awaiting requested review from malilex malilex is a code owner

+1 more reviewer

@luizasok luizasok luizasok approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

compose

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@SerraMorec @salutedevs @raininforest @luizasok @angirb