Skip to content

PLASMA-8122: feat(sdds-alibs/homeds): bubble was implemented - #913

Open
raininforest wants to merge 2 commits into
developfrom
feature/bubble
Open

PLASMA-8122: feat(sdds-alibs/homeds): bubble was implemented#913
raininforest wants to merge 2 commits into
developfrom
feature/bubble

Conversation

@raininforest

Copy link
Copy Markdown
Collaborator

plasma-homeds-compose

Bubble

  • Реализован компонент Bubble.

What/why changed

Реализован компонент Bubble

Screen_recording_20260911_113505.webm

@raininforest raininforest self-assigned this Sep 11, 2026
@raininforest raininforest added the enhancement New feature or request label Sep 11, 2026
@coderabbitai

coderabbitai Bot commented Sep 11, 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: Advanced

Run ID: e7478c90-b68b-4368-ada4-98920dd3ed26

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

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.

@raininforest raininforest changed the title fix(sdds-alibs/homeds): bubble was implemented PLASMA-8122: fix(sdds-alibs/homeds): bubble was implemented Sep 11, 2026
@raininforest raininforest changed the title PLASMA-8122: fix(sdds-alibs/homeds): bubble was implemented PLASMA-8122: feat(sdds-alibs/homeds): bubble was implemented Sep 11, 2026
Box(
modifier = modifier
.size(circleSize)
.onGloballyPositioned { coordinates ->

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.

onGloballyPositioned в списках может влиять на производительность. Поговори с Никитой, у него была похожая ситуация с Popover, он делал ресерч.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Сделал фикс. Вынес координаты в обычный (не-Snapshot) холдер MutableRef, а в ключах LaunchedEffect оставил только rootCoordinates (меняется редко — поворот/ресайз хоста) — видимость по-прежнему пересчитывается синхронно внутри onGloballyPositioned, но без лишней рекомпозиции и пересоздания корутины на каждый кадр

@salutedevs salutedevs left a comment

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.

  • Мне кажется, что текст в попапе нужно клипать при анимации. Сейчас видно, что он отрисовывается раньше, чем форма попапа растет.
  • Заметно, что звезда превращается в круг альфой. Выглядит не очень. Мы не можем анимировать трансформацию шейпа?

@salutedevs

salutedevs commented Sep 11, 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

@salutedevs

Copy link
Copy Markdown
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compose enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants