CashAdv / CashADV

0 stars 10 forks source link

Feature/51 code confirmation view #85

Closed artwist-polyakov closed 6 months ago

alexxk2 commented 7 months ago

Привет, Прикрутил твой пинкод к своему фрагменту, всё работает, но с нюансами) Сначала опишу словами, потом попробую приложить скрины и видео.

  1. Надо что-то сделать, чтобы разделители ячеек сами равномерно разделяли пространство вью(горизонтально). Сейчас ты ставишь symbolsSpacing, но это не работает как надо. Нам нужно, чтобы при вводе codeLength и установке ширины вью, она сама ставила разделители куда надо и делала одинаковые ячейки.
  2. Ввел цифры и пытаешься сразу их удалить - вью начинает их удалять только после нескольких нажатий, как будто там есть еще парочка невидимых знаков, которые надо удалить (на видео).
  3. Непонятно как сделать так, чтобы черточки (- - - -) вот эти, были расположены по центру своих ячеек, сейчас у них как будто gravity = start
  4. Опять про черточки, они расположены очень близко к середине вью по вертикали, на дизайне они ниже.
  5. Не понял как ты сохраняешь стейт вью, но он как-то сохраняется)) Я в своем делал onSaveInstanceState и onRestoreInstanceState.
  6. Может быть еще можно перед вью сделать описание и как использовать атрибуты в java doc стиле. Будет удобно.

Это пока все что получилось найти. Мб напишу что-то еще потом. Получилось очень круто и и главное

https://github.com/CashAdv/CashADV/assets/96295239/7d852770-6f54-4a19-abf9-d422ec717904

работает))

avanisimov commented 7 months ago

Нужна превьюха в UiKit

artwist-polyakov commented 6 months ago

Нужна превьюха в UiKit

Тут готово @avanisimov

artwist-polyakov commented 6 months ago
  1. Надо что-то сделать, чтобы разделители ячеек сами равномерно разделяли пространство вью(горизонтально). Сейчас ты ставишь symbolsSpacing, но это не работает как надо. Нам нужно, чтобы при вводе codeLength и установке ширины вью, она сама ставила разделители куда надо и делала одинаковые ячейки.

@alexxk2 привет, тут я придерживался логики, что у нас ячейка 75 на 40 dp. Этого гарантированно хватит чтобы поместиться в 320 пикселов ширины. Вмещается даже на Фолде. image

artwist-polyakov commented 6 months ago

2. Ввел цифры и пытаешься сразу их удалить - вью начинает их удалять только после нескольких нажатий, как будто там есть еще парочка невидимых знаков, которые надо удалить (на видео).

@alexxk2 тут у меня не было ошибки, но когда я навесил коллбек чтобы тот выводил тост (см демо в проекте) то на удалении коллбек не вызывался. Если ты пытался в коллбеке устанавливать код, то в этом может быть причина глюка.

сет код нужен только при восстановлении фрагмента, но не для установки его каждый раз при нажатии на кнопку

хотя сейчас возможно что и при нажатии на кнопку будет ок — я навесил коллбек на удаление

artwist-polyakov commented 6 months ago
  • Непонятно как сделать так, чтобы черточки (- - - -) вот эти, были расположены по центру своих ячеек, сейчас у них как будто gravity = start
  • Опять про черточки, они расположены очень близко к середине вью по вертикали, на дизайне они ниже.

Тут разобрались еще неделю назад

artwist-polyakov commented 6 months ago

5. Не понял как ты сохраняешь стейт вью, но он как-то сохраняется)) Я в своем делал onSaveInstanceState и onRestoreInstanceState.

не сохраняется оно) либо магия. при восстановлении из фона надо засеттлить код.

artwist-polyakov commented 6 months ago

6. Может быть еще можно перед вью сделать описание и как использовать атрибуты в java doc стиле. Будет удобно.

done

CleanShot 2024-04-21 at 20 03 40