Skip to content

Conversation

AhmedAmineZr
Copy link
Member

@AhmedAmineZr AhmedAmineZr commented Aug 26, 2025

Note: Please transform - [ ] into - (NA) in the description when things are not applicable

Related issues

Description

Motivation & Context

Types of change

  • Bug fix (non-breaking which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

Previews

Checklist

Contribution

Accessibility

  • My change follows accessibility good practices

Design

  • My change respects the design guidelines of Orange Unified Design System

Development

  • My change follows the developer guide
  • I have added unit tests to cover my changes (optional)

Documentation

  • My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

  • Manually test (dark mode, RTL, landscape display, tablet)
  • Documentation has been updated if relevant
  • Design review
  • A11y review
  • Internal files have been updated if relevant (THIRD_PARTY, NOTICE)
  • changelog.md has been updated respecting keep a changelog rules and referencing the issue

@AhmedAmineZr AhmedAmineZr linked an issue Aug 26, 2025 that may be closed by this pull request
@AhmedAmineZr AhmedAmineZr self-assigned this Aug 26, 2025
@AhmedAmineZr AhmedAmineZr added ⚛️ component 📖 documentation Improvements or additions to documentation 📱 demo app 🧰 library ⭐ v1.0 Tickets needed for mobile v1.0 release labels Aug 26, 2025
@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch 3 times, most recently from 2e8cbe5 to bcb1c5c Compare September 3, 2025 21:44
@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch 3 times, most recently from 2a3b0db to 661e422 Compare September 19, 2025 16:17
@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch 2 times, most recently from 7d86399 to 297f20e Compare September 24, 2025 11:02
@AhmedAmineZr AhmedAmineZr marked this pull request as ready for review September 24, 2025 13:32
@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch 2 times, most recently from 46e744e to 8f973fe Compare September 25, 2025 13:51
@Tayebsed93
Copy link
Collaborator

Tayebsed93 commented Sep 26, 2025

hello @AhmedAmineZr,

  1. Always allow entering 0, even when the number starts with a +, to support users who make mistakes.
Capture d’écran 2025-09-26 à 11 54 11
  1. For the refactoring, create a form_fields folder and put everything inside it. The shared files will be called FormField.
form_fields/
  text_input/
    text_input.dart
  phone_number/
    phone_number_input.dart
  1. We are getting this error:
======== Exception caught by services library ======================================================
The following assertion was thrown during method call TextInputClient.updateEditingState:
Range start 17 is out of text of length 8
'package:flutter/src/services/text_input.dart':
Failed assertion: line 1130 pos 7: 'range.start >= 0 && range.start <= text.length'
Capture d’écran 2025-09-26 à 11 52 31
  1. In the initial settings, the country selector is enabled with enabled = true by default.
Capture d’écran 2025-09-26 à 11 51 26

@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch 4 times, most recently from 1de2658 to 8e261f4 Compare October 2, 2025 16:18
@AhmedAmineZr AhmedAmineZr force-pushed the 326-create-component---phone-number-input branch from 07f4845 to ae1f699 Compare October 15, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚛️ component 📱 demo app 📖 documentation Improvements or additions to documentation 🧰 library ⭐ v1.0 Tickets needed for mobile v1.0 release ⌛ waiting for validations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create component - Phone Number Input

3 participants