Skip to content

Conversation

@Avisrara74
Copy link

Кратко сформулируйте суть доработки
Нужна возможность вообще скрыть флажок выбора страны. У бизнеса требование - не отображать флаг для стран с кодом +7 (казахстан, россия, южная осетия, абхазия) в многошаге, по аналогии с АМ. Компонент phone-input жестко захардкожен только на +7 для России и имеет множество ограничений которые вскрылись на тестировании.
Пока выкладываю ПР чтобы ваши разрабы посмотрели, когда "окните", добавлю тесты и changeset
Отчет о тестировании: https://confluence.moscow.alfaintra.net/pages/viewpage.action?pageId=2908432147

Чек лист

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

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

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

Было:
PhoneInputBasicInternational-desktop

Стало:
PhoneInputBasicRussian-desktop

@changeset-bot
Copy link

changeset-bot bot commented Dec 2, 2025

⚠️ No Changeset found

Latest commit: 5de469e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@coveralls
Copy link

Pull Request Test Coverage Report for Build 19851607232

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.01%) to 79.694%

Totals Coverage Status
Change from base Build 19334456547: -0.01%
Covered Lines: 9668
Relevant Lines: 11364

💛 - Coveralls

@fulcanellee
Copy link
Collaborator

@Oladii посмотри пожалуйста

@Oladii
Copy link
Contributor

Oladii commented Dec 4, 2025

а демки нет, как смотрец? В целом ок, нужно убедиться что при скрытии левых аддонов отступ от текста до левого края инпута такой же как в обычном инпуте (проверить для всех поддерживаемых размеров)

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.

4 participants