Skip to content

Conversation

@Tayebsed93
Copy link
Collaborator

@Tayebsed93 Tayebsed93 commented Sep 30, 2025

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

Related issues

#397

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

@Tayebsed93 Tayebsed93 self-assigned this Sep 30, 2025
@Tayebsed93 Tayebsed93 linked an issue Sep 30, 2025 that may be closed by this pull request
@Tayebsed93 Tayebsed93 changed the title Component : Password input feat : Component Password input Sep 30, 2025
@Tayebsed93 Tayebsed93 changed the title feat : Component Password input feat : component password input Sep 30, 2025
@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch 2 times, most recently from b949d71 to 97bec3c Compare October 3, 2025 16:12
@Tayebsed93 Tayebsed93 marked this pull request as ready for review October 6, 2025 15:56
@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch from 97bec3c to 7be6af8 Compare October 6, 2025 16:03
@AhmedAmineZr AhmedAmineZr self-requested a review October 7, 2025 12:49
@nouha06 nouha06 self-requested a review October 8, 2025 07:49
@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch 4 times, most recently from 002d36a to 29db125 Compare October 9, 2025 11:18
@AhmedAmineZr
Copy link
Member

@Tayebsed93 Bravo pour le travaille réalisé , j'ai quelque remarque de plus :
1- Quand on affiche le Password dans le 1er controller le 2éme doit suivre puisque :
Capture d’écran 2025-10-10 à 11 31 56

2- dans les costumization on une valeur de Hidden Password qui manque dans le App , je sais que y a l'icône qui fait le même travaille mais je pense qu'on doit ajouté
Capture d’écran 2025-10-10 à 11 38 41

Good Job

@Tayebsed93
Copy link
Collaborator Author

@Tayebsed93 Bravo pour le travaille réalisé , j'ai quelque remarque de plus : 1- Quand on affiche le Password dans le 1er controller le 2éme doit suivre puisque : Capture d’écran 2025-10-10 à 11 31 56

2- dans les costumization on une valeur de Hidden Password qui manque dans le App , je sais que y a l'icône qui fait le même travaille mais je pense qu'on doit ajouté Capture d’écran 2025-10-10 à 11 38 41

Good Job

@AhmedAmineZr Merci pour ta remarque. Nous n'avons pas pas de callback (car je n'en vois pas l'utilité); la lib est fermé donc on ne peux pas récupérer l'état du bouton (hidden/visible)
Je ne pense pas que ce soit utile d'ouvrir l'api

@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch from 29db125 to 8f58a56 Compare October 10, 2025 12:05
@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch 2 times, most recently from 69955ac to b70de8c Compare October 20, 2025 15:34
@Tayebsed93 Tayebsed93 force-pushed the 397-create-component---password-input branch from 3548aa4 to ce37daa Compare November 4, 2025 10:58
@nouha06
Copy link
Member

nouha06 commented Nov 4, 2025

Merci pour ce travaille réaliser pour le composant
J’ai seulement quelques remarques mineures :
Ici, lorsqu’un utilisateur saisit un long texte dans le placeholder, le champ s’agrandit, mais lorsqu’il saisit son mot de passe, le champ reste étendu.
image

et les cas d'erreurs ne sont pas traiter dans app de demo? genre quand utilisateur saisie un mot de passe less than 8 characters ou quand il saisie un champs vide ...

@nouha06
Copy link
Member

nouha06 commented Nov 4, 2025

Merci pour ce travaille réaliser pour le composant J’ai seulement quelques remarques mineures : Ici, lorsqu’un utilisateur saisit un long texte dans le placeholder, le champ s’agrandit, mais lorsqu’il saisit son mot de passe, le champ reste étendu. image

et les cas d'erreurs ne sont pas traiter dans app de demo? genre quand utilisateur saisie un mot de passe less than 8 characters ou quand il saisie un champs vide ...

resolved

nouha06 and others added 9 commits November 4, 2025 17:13
# Conflicts:
#	app/CHANGELOG.md
#	app/lib/l10n/gen/ouds_flutter_app_localizations.dart
#	app/lib/l10n/gen/ouds_flutter_app_localizations_ar.dart
#	app/lib/l10n/gen/ouds_flutter_app_localizations_en.dart
#	app/lib/l10n/ouds_flutter_ar.arb
#	app/lib/l10n/ouds_flutter_en.arb
#	app/lib/ui/components/components.dart
#	app/lib/ui/components/form_input/form_fields_customization_utils.dart
#	app/lib/ui/components/form_input/text_input/text_input_demo_screen.dart
#	app/lib/ui/utilities/customizable/customizable_textfield.dart
#	ouds_core/CHANGELOG.md
#	ouds_core/lib/components/form_input/ouds_text_input.dart
#	ouds_core/lib/l10n/gen/ouds_localizations.dart
#	ouds_core/lib/l10n/gen/ouds_localizations_ar.dart
#	ouds_core/lib/l10n/gen/ouds_localizations_en.dart
#	ouds_core/lib/l10n/ouds_flutter_ar.arb
#	ouds_core/lib/l10n/ouds_flutter_en.arb
@Tayebsed93 Tayebsed93 merged commit 711b276 into develop Nov 6, 2025
3 checks passed
@Tayebsed93 Tayebsed93 deleted the 397-create-component---password-input branch November 6, 2025 14:59
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.

Create component - Password Input

3 participants