PLASMA-8122: feat(sdds-alibs/homeds): bubble was implemented - #913
PLASMA-8122: feat(sdds-alibs/homeds): bubble was implemented#913raininforest wants to merge 2 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
| Box( | ||
| modifier = modifier | ||
| .size(circleSize) | ||
| .onGloballyPositioned { coordinates -> |
There was a problem hiding this comment.
onGloballyPositioned в списках может влиять на производительность. Поговори с Никитой, у него была похожая ситуация с Popover, он делал ресерч.
There was a problem hiding this comment.
Сделал фикс. Вынес координаты в обычный (не-Snapshot) холдер MutableRef, а в ключах LaunchedEffect оставил только rootCoordinates (меняется редко — поворот/ресайз хоста) — видимость по-прежнему пересчитывается синхронно внутри onGloballyPositioned, но без лишней рекомпозиции и пересоздания корутины на каждый кадр
salutedevs
left a comment
There was a problem hiding this comment.
- Мне кажется, что текст в попапе нужно клипать при анимации. Сейчас видно, что он отрисовывается раньше, чем форма попапа растет.
- Заметно, что звезда превращается в круг альфой. Выглядит не очень. Мы не можем анимировать трансформацию шейпа?
📦 Собранные Debug APK📲 Как установить APK на Android-устройство:
💻 Установка через USB (ADB):
|
|
📘Артефакты документации опубликованы: |
f648149 to
47a32dd
Compare
plasma-homeds-compose
Bubble
What/why changed
Реализован компонент Bubble
Screen_recording_20260911_113505.webm