fix(budget): UI compacte Material 3 pour la saisie des montants en mode Split #14

Merged

alexis merged 5 commits from fix/coversheet-split-ui-m3 into main 2026-09-03 02:21:16 +02:00

Conversation 2 Commits 5 Files changed 2 +48 -11

alexis commented 2026-09-03 02:01:19 +02:00

Owner

Copy link

Résumé des ajustements

• UI Material 3 compacte : Remplacement du grand champ (56dp de haut) par un conteneur inline épuré de 38dp avec , s'intégrant harmonieusement avec le « Inclus / Ajouter ».
• Règle de workflow : Ajout dans de la règle stricte d'interdiction de merge automatique afin de toujours laisser la PR ouverte pour test réel sur téléphone avant validation.
• Build & Déploiement : Validé et déployé sur le téléphone via ADB WiFi.

### Résumé des ajustements - **UI Material 3 compacte** : Remplacement du grand champ (56dp de haut) par un conteneur inline épuré de 38dp avec , s'intégrant harmonieusement avec le « Inclus / Ajouter ». - **Règle de workflow** : Ajout dans de la règle stricte **d'interdiction de merge automatique** afin de toujours laisser la PR ouverte pour test réel sur téléphone avant validation. - **Build & Déploiement** : Validé et déployé sur le téléphone via ADB WiFi.

alexis added 1 commit 2026-09-03 02:01:20 +02:00

fix(budget): UI compacte Material 3 pour la saisie des montants en mode Split ... b7678cb34d

- Remplacement du OutlinedTextField trop haut (56dp) par un Surface compact (38dp) + BasicTextField
- Typographie et espacements alignés sur les FilterChips M3 Expressive
- WORKFLOW.md: règle stricte d'interdiction de merge automatique sans validation utilisateur préalable

alexis added 1 commit 2026-09-03 02:08:28 +02:00

docs(workflow): test programmatique obligatoire sur appareil réel via ADB pour chaque PR 36a82d418e

alexis added 1 commit 2026-09-03 02:10:37 +02:00

docs(workflow): ajout de la capture d'écran et validation visuelle obligatoire sur device réel 25d280ea1d

alexis added 1 commit 2026-09-03 02:15:46 +02:00

fix(budget): centrage optique vertical et horizontal du montant dans la box de split a67d196ca1

alexis added 1 commit 2026-09-03 02:19:41 +02:00

fix(budget): désactivation du clic sur la ligne entière en mode Split (seul le bouton Ajouter réagit) 660ef7c426

alexis merged commit d14e3fcc07 into main 2026-09-03 02:21:16 +02:00

alexis deleted branch fix/coversheet-split-ui-m3 2026-09-03 02:21:16 +02:00

alexis referenced this pull request from a commit 2026-09-03 02:21:18 +02:00

Merge pull request 'fix(budget): UI compacte Material 3 pour la saisie des montants en mode Split' (#14) from fix/coversheet-split-ui-m3 into main

alexis commented 2026-09-05 04:01:54 +02:00

Author

Owner

Copy link

🤖 AI Review (opencode-free via 9router)

Résumé (2 lignes)

Modification UI d'un champ texte dans CoverSourceRow pour un style plus compact et Material 3. Ajout de directives de workflow pour le test automatisé et la validation visuelle sur téléphone.

Problèmes

1. Fichier docs/WORKFLOW.md : Diff affiché vide/incomplet (impossible de valider le contenu ajouté). Vérifier le merge.
2. BasicTextField non importé : androidx.compose.foundation.text.BasicTextField utilisé en qualifié complet, ce qui fonctionne mais est inhabituel dans un codebase qui utilise déjà des imports. Aucun impact fonctionnel.
3. surfaceContainerHigh : Assurer que le thème M3 du projet expose bien cette couleur (disponible depuis M3 1.1).

Suggestions

1. Extraire Modifier.width(96.dp).height(38.dp) dans un Modifier commun si réutilisé.
2. Ajouter @Preview pour CoverSourceRow avec splitMode = true/false afin de vérifier visuellement sans émulateur.
3. Ajouter un test unitaire minimal : assert(splitMode) bloque bien le clickable (si splitMode est une donnée, un simple test de logique suffit).
4. Valider que le Spacer(Modifier.width(2.dp)) est le bon espacement (penser à 8.dp grid Material 3).
5. Envisager un OutlinedTextField custom au lieu de BasicTextField si un style Material 3 est souhaité (moins de code, natif M3).

Verdict : LGTM

<!-- ai-review --> ## 🤖 AI Review (`opencode-free` via 9router) ## Résumé (2 lignes) Modification UI d'un champ texte dans `CoverSourceRow` pour un style plus compact et Material 3. Ajout de directives de workflow pour le test automatisé et la validation visuelle sur téléphone. ## Problèmes 1. **Fichier `docs/WORKFLOW.md`** : Diff affiché vide/incomplet (impossible de valider le contenu ajouté). Vérifier le merge. 2. **`BasicTextField` non importé** : `androidx.compose.foundation.text.BasicTextField` utilisé en qualifié complet, ce qui fonctionne mais est inhabituel dans un codebase qui utilise déjà des imports. Aucun impact fonctionnel. 3. **`surfaceContainerHigh`** : Assurer que le thème M3 du projet expose bien cette couleur (disponible depuis M3 1.1). ## Suggestions 1. Extraire `Modifier.width(96.dp).height(38.dp)` dans un `Modifier` commun si réutilisé. 2. Ajouter `@Preview` pour `CoverSourceRow` avec `splitMode = true/false` afin de vérifier visuellement sans émulateur. 3. Ajouter un test unitaire minimal : `assert(splitMode)` bloque bien le `clickable` (si `splitMode` est une donnée, un simple test de logique suffit). 4. Valider que le `Spacer(Modifier.width(2.dp))` est le bon espacement (penser à `8.dp` grid Material 3). 5. Envisager un `OutlinedTextField` custom au lieu de `BasicTextField` si un style Material 3 est souhaité (moins de code, natif M3). ## Verdict : **LGTM**

alexis commented 2026-09-05 17:38:20 +02:00

Author

Owner

Copy link

Hey @alexis — nice focused PR. The three BudgetScreen changes (conditional row click, custom text field, chip sizing) are clean and the WORKFLOW doc additions are well-structured.

🟢 The mandatory testing rule in docs/WORKFLOW.md (lines 121-127) is written as a numbered sub-list with concrete ADB commands, making it actionable rather than vague guidance.

Minor note (non-blocking): BudgetScreen.kt line 1532 uses a fully-qualified androidx.compose.foundation.text.BasicTextField instead of a top-level import. Works fine, just a style inconsistency with the rest of the file — low priority cleanup if you ever revisit.

Looks good to merge.

Hey @alexis — nice focused PR. The three BudgetScreen changes (conditional row click, custom text field, chip sizing) are clean and the WORKFLOW doc additions are well-structured. 🟢 The mandatory testing rule in `docs/WORKFLOW.md` (lines 121-127) is written as a numbered sub-list with concrete ADB commands, making it actionable rather than vague guidance. **Minor note (non-blocking):** `BudgetScreen.kt` line 1532 uses a fully-qualified `androidx.compose.foundation.text.BasicTextField` instead of a top-level import. Works fine, just a style inconsistency with the rest of the file — low priority cleanup if you ever revisit. Looks good to merge.

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

alexis/demo_m3!14

No description provided.