PLASMA-6379: NavigationBar was implemented for xml#675
PLASMA-6379: NavigationBar was implemented for xml#675SerraMorec wants to merge 2 commits intodevelopfrom
Conversation
SerraMorec
requested review from
angirb,
malilex and
raininforest
as code owners
|
📘Артефакты документации опубликованы: |
📦 Собранные APK📲 Как установить APK на Android-устройство:
💻 Установка через USB (ADB):
|
| heightMeasureSpec, | ||
| 0, | ||
| ) | ||
| widthUsed += centerContent.measuredWidth + verPaddings |
There was a problem hiding this comment.
почему тут вертикальные паддинги, а не горизонтальные?
There was a problem hiding this comment.
да, это ошибка, спасибо, поправил
| 0, | ||
| ) | ||
| widthUsed += centerContent.measuredWidth + verPaddings | ||
| val heightUsed = horPaddings + |
There was a problem hiding this comment.
а здесь горизонтальные вместо вертикальных
There was a problem hiding this comment.
да, это ошибка, спасибо, поправил
SerraMorec
force-pushed
the
feature/plasma-6379-navbar
branch
from
f6bbf08 to
1e54d19
Compare
| val actionStartH = if (actionStart.isVisible) actionStart.measuredHeight else 0 | ||
|
|
||
| val actionEndW = if (actionEnd.isVisible) actionEnd.measuredWidth else 0 | ||
| val actionEndH = if (actionEnd.isVisible) actionEnd.measuredHeight else 0 |
There was a problem hiding this comment.
Если visibility == INVISIBLE, то ширина/высота не будут учитываться, а по идее должны.
Или я что-то упускаю?
|
|
||
| if (centeringStrategy == CenteringStrategy.RELATIVE) { | ||
| var top = viewTop(actionStartH) | ||
| if (actionStart.isVisible) { |
There was a problem hiding this comment.
и тут, если visibility == View.INVISIBLE, то вьюшка должна быть расположена. Судя по коду, это не так
| private var titleAppearances: StyleStateList? = null | ||
| private var currentTitleAppearance: Int = 0 | ||
| private var currentDescriptionAppearance: Int = 0 | ||
| private var titleColors: ColorStateList? = null |
There was a problem hiding this comment.
Мы точно можем ограничиться только ColorStateList?
У нас есть ColorValueStateList, который поддерживает токены градиента.
Думаю, что нужно добавить для всех свойств цвета в этом классе ColorValueStateList
| * Роль назначается автоматически как [NavigationBarContent.TITLE] | ||
| */ | ||
| open fun setTitle(title: View, params: NavigationBarLayoutParams? = null) { | ||
| universalAddView(title, params, NavigationBarContent.TITLE) |
There was a problem hiding this comment.
Что произойдет, если пользователь установит два title? я правильно понял, что внутри setRoleLink удалится старая вьюшка и добавиться новая?
4 participants
sdds-uikit
NavigationBar
sdds-serv
NavigationBar
plasma-sd-service
NavigationBar
File
What/why changed
Реализован новый компонент NavigationBar, написана документация, реализован генератор.
2026-02-16.09.59.27.mov
2026-02-16.10.00.13.mov
2026-02-16.10.01.01.mov