Skip to content

PLASMA-7414: Motion API was added in TextSkeleton and RectSkeleton#820

Open
SerraMorec wants to merge 1 commit into
developfrom
feature/plasma-7414
Open

PLASMA-7414: Motion API was added in TextSkeleton and RectSkeleton#820
SerraMorec wants to merge 1 commit into
developfrom
feature/plasma-7414

Conversation

@SerraMorec

Copy link
Copy Markdown
Collaborator

sdds-uikit-compose

TextSkeleton

  • В компонент добавлено motion API

RectSkeleton

  • В компонент добавлено motion API

What/why changed

Для компонента TextSkeleton и RectSkeleton добавлены Motion и MotionStyle, с помощью которых можно настроить анимацию изменения параметров цвета и типографики. Стили переведен на StatefulValue.

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 635ffbf3-33d9-48e2-bb33-932637a75fcc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/plasma-7414

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@SerraMorec SerraMorec force-pushed the feature/plasma-7414 branch from 5f6b4ee to bc78447 Compare June 15, 2026 05:22
@salutedevs

salutedevs commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

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

@salutedevs

salutedevs commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

📦 Собранные Debug 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

brush: StatefulValue<Brush> = style.gradient,
shape: StatefulValue<Shape> = style.shapes,
motion: Motion<RectSkeletonMotionStyle> = rememberRectSkeletonMotion(),
transition: InfiniteTransition = rememberInfiniteTransition(),

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.

А зачем новая перегрузка, можно добавить motion в существующие перегрузки? Они будут конфликтовать, так как ни одна не требует явно указать какой-то из параметров.

} else {
Modifier.blink(
color = brushValue.value,
color = (brushValue as SolidColor).value,

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.

можно сделать перегрузки для Modifier.blink и Modifier.brush с лямбдой для brush и color. С обратно совместимостью, конечно.

brush: StatefulValue<Brush> = style.gradient,
shape: StatefulValue<Shape> = style.shapes,
transition: InfiniteTransition = rememberInfiniteTransition(),
motion: Motion<TextSkeletonMotionStyle> = rememberTextSkeletonMotion(),

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.

конфликт перегрузок. Можно добавить motion в существующую функцию.

@SerraMorec SerraMorec force-pushed the feature/plasma-7414 branch from bc78447 to afb1cbb Compare June 17, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants