Skip to content

Conversation

@VladislavNsk
Copy link
Contributor

@VladislavNsk VladislavNsk commented Oct 26, 2025

Кратко сформулируйте суть доработки

Фикс нескольких багов, когда передан props view=withZeroMinorPart:

  • пользователь начинает вводить целое число и каретка перескакивает в конец поля ввода, за десятичную часть
  • пользователь начинает удалять десятичную часть, то дойдя до целой части, воспроизводился баг из п1
  • начальное значение 0,00, если начать вводить число, ничего не происходило
  • начальное значение 0,00, если поставить каретку между "0" и разделителем "," и начать вводить 0, то могли добавить ноль и получалось следующее значение: "000,00"

Чек лист

  • Задача сформулирована и описана в JIRA
  • В названии ветки есть айдишник задачи в JIRA (fix/DS-1234), ссылку прикреплять не надо
  • У реквеста осмысленное название feat(...) или fix(...) по conventional commits (https://www.conventionalcommits.org)
  • Код покрыт тестами и протестирован в различных браузерах
  • Добавленные пропсы добавлены в демки и описаны в документации
  • К реквесту добавлен changeset

Если есть визуальные изменения

  • Прикреплено изображение было/стало

Код для воспроизведения в песочнице

function Example() {
    const [value, setValue] = React.useState(0);
    
    const handleChange = (_, payload) => {
        setValue(payload.value);
    };
    
    return (
        <AmountInput
            defaultValue={value}
            placeholder='Введите сумму'
            labelView='outer'
            label='Сумма'
            minority={100}
            bold={false}
            currency='RUR'
            codeFormat='letter'
            view='withZeroMinorPart'
            value={value}
            onChange={handleChange}
        />
    );
}
render(
    <Example />
)

@changeset-bot
Copy link

changeset-bot bot commented Oct 26, 2025

🦋 Changeset detected

Latest commit: fa31132

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@alfalab/core-components-amount-input Minor
@alfalab/core-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coveralls
Copy link

Pull Request Test Coverage Report for Build 18815274068

Details

  • 16 of 16 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 79.746%

Totals Coverage Status
Change from base Build 18726941599: 0.05%
Covered Lines: 9655
Relevant Lines: 11349

💛 - Coveralls

@VladislavNsk VladislavNsk changed the title Fix/ds 13498 fix(amount-input): fix typing and removing digits when input equel zero with minor digits Oct 26, 2025
@github-actions
Copy link
Contributor

Demo build

https://core-ds.github.io/core-components/1928

@fulcanellee
Copy link
Contributor

Некоторую работу по этому компоненту недавно проводил @hextion, нужно с ним засинкаться, потому что некоторые правки могут уже лежать в другой ветке

@fulcanellee
Copy link
Contributor

PR не актуален. Работа проделана в другой ветке

@hextion
Copy link
Contributor

hextion commented Jan 22, 2026

#1961

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants