12 create custom view movie selection #23
Conversation
Класс содержит логику селектора Метод обновления состояния селектора возвращает коллбэк Предусмотрено использование изображений из разных источников (для кэширования изображений используется Coil)
Удаляет методы установки изображений не из сети: - удаляет атрибут для установки изображения (изображение можно установить только через один метод) - оставляет метод для загрузки изображения по url Форматирует код в соответствии с контрактом detekt
|
Направь плз этот ПР в ветку 10_create_custom_view_movie_evaluation, чтобы дифф более читаемый был |
|
| placeholder(R.drawable.placeholder_img) | ||
| transformations( | ||
| RoundedCornersTransformation( | ||
| radius = resources.getDimensionPixelSize(R.dimen.card_radius_16).toFloat() |
There was a problem hiding this comment.
Чтобы толщина зеленой/белой линии вокруг вьюхи была одного размера в углах, то внешний и внутренний радиус можно сделать отличными на величину ширины этой линии.
То есть можно или здесь радиус уменьшить на 8 (внутренний радиус), либо увеличить внешний радиус на 8.
Еще трансформация у coil работает только к основному изображению, то есть в плейсхолдерах, которые у нас потом будут, нужно чтобы углы уже были скруглены.
There was a problem hiding this comment.
Спасибо за замечания, тут нужно было не апрувить))
По поводу трансформации убил 2 часа времени чтобы заставить скруглить углы на плейсхолдере и на ошибках.
There was a problem hiding this comment.
Пофиксил радиус скругления
| @@ -0,0 +1,106 @@ | |||
| package com.davai.uikit | |||
There was a problem hiding this comment.
к делу не относится конечно, но у нас пакеты опять по-разному называются))
в app com.davay
There was a problem hiding this comment.
я снова накосячил, можем выделить один ПР на то, чтобы переименовать пакеты
|
Пофиксил скругления углов для любых изображений (apceholder и error) |
Добавляет картинку в один из сэмплов
…custom_view_movie_selection # Conflicts: # uikit/src/main/res/drawable/error_img.xml # uikit/src/main/res/drawable/placeholder_img.xml # uikit/src/main/res/values/strings.xml # uikit_sample/src/main/AndroidManifest.xml
Зарезолвил |
Добавляет кастомную view селектор подборки
https://www.figma.com/file/18yNJmNYwWsexJku2SpWWE/%D0%94%D0%B0%D0%B2%D0%B0%D0%B9-%D0%BF%D0%BE%D1%81%D0%BC%D0%BE%D1%82%D1%80%D0%B8%D0%BC?type=design&node-id=156-2370&mode=design&t=5JPjVoKX7QkDlr0c-4