Repository navigation
fix(mobile): garder la tooltip du départ sous la barre du haut - #629
Merged
Merged
Conversation
Fitting bounds, the native map adds the padding the camera already has to the one given (measured on Android, same deprecated API on iOS): the route landed elsewhere than screenPointOnFit expects, and the tooltip of the start left the screen. The camera now takes the centre and zoom of the fit, computed in the same projection, with the full padding. (cherry picked from commit bab02fc, branch issue/305-route-frame-camera) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The padding of the camera gave the upper half to the tooltip but capped its room to leave the route a touch target above the sheet: with a tall summary, the tooltip of the start passed under the search pill and the chips. The highest step now always sits the measured bar, a 16 point gap and the measured tooltip below the top of the map, and the route keeps whatever band is left above the sheet, the map zooming out until it fits. The sheet gives way only when no band is left at all. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Collaborator
Author
|
Plan et décisions techniques :
|
This was referenced Oct 22, 2026
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Résultat et problème
Quand on sélectionne un parcours, depuis un marqueur ou un badge « 1 », la tooltip du départ passait derrière la pilule de recherche et les chips (constat de Paul, Redmi Note 9, 2026-10-22).
Cause : celle de #305, cas déjà couvert par son correctif, sur deux étages.
boundsest faux sur les deux systèmes. Dans @rnmapbox/maps 10.3.5, la marge demandée s'ajoute à celle que la caméra a déjà. Or la vue d'accueil a une marge, puisqu'elle suit la barre du haut et la sheet. Le départ arrive donc plus haut quescreenPointOnFitne le prévoit, sous la barre, et décalé à droite. Les mesures sont dans #305. Je l'ai reproduit aujourd'hui sur iOS à 100 % depuismain(capture « avant »).routeFramePaddingplafonnait la réserve haute pour laisser 44 pt au tracé au-dessus de la sheet. Avec un résumé haut (texte agrandi), la tooltip passait donc sous la barre par construction.Correctif, en deux commits :
fix(mobile): frame a selected route with its centre and zoom: c'est le commitbab02fcde la brancheissue/305-route-frame-camera, repris par cherry-pick avec son auteur. Je l'ai rebasé surmainen résolvant les conflits avectooltipWidthettooltipPlacement(D-041). La caméra prend le centre et le zoom calculés par l'app (cameraOnFit, même projection quescreenPointOnFit) avec la marge complète, sansbounds.fix(mobile): keep the tooltip of a selected route under the top barapplique la règle précisée par Paul le 2026-10-22 : la tooltip du départ et toutes les étapes tiennent dans la zone visible, entre la barre de recherche et les pilules (+ 16 px) en haut et le résumé en bas. Le zoom n'est plus une contrainte : des pastilles qui se chevauchent sont acceptées, l'utilisateur peut zoomer.Limites restantes
main. Je l'ai déposé dans l'Inbox.Fixes #626
Écrans : E-01, E-04
Dépendance : aucune. Reprend le commit
bab02fcde #305 sans fermer #305.Comment vérifier
pnpm turbo run lint typecheck --force,prettier --check .etpnpm --filter mobile test(280 tests) verts en local. Tests de la carte : 93, dontcameraOnFitet les nouveaux cas derouteFramePadding(la tooltip reste sous la barre quand la sheet et la tooltip sont hautes, le tracé est cadré dans la bande qui reste, la sheet ne cède que sans bande).main(avant)Sur ces captures, « De la Cité à Montparnasse » a 5 étapes. Avant le correctif, la tooltip passe sous les chips et sort à droite, et la dernière étape est coupée. Après, la tooltip est entière sous les chips et les 5 étapes sont au-dessus du résumé. En XXXL, la tooltip reste sous les chips, et les étapes sont regroupées au-dessus du résumé mais visibles. Le badge « 1 » donne le même cadrage qu'un tap sur le marqueur. Pour faire apparaître les badges en local, j'ai abaissé temporairement le réglage
search.cluster_area_km2dans la base locale, puis je l'ai retiré.Rejouer sur simulateur depuis
main, après la fusion, avec Docker etwidoo-app-db-1démarrés :Accessibilité inchangée : aucun composant ni libellé n'est modifié, seul le cadrage de la caméra change. Texte agrandi testé en XXXL sur iOS. L'essai à 150 % sur Android reste à faire (voir plus haut).
Formatage mécanique : non
Risque
Niveau : 1. Le changement porte seulement sur le cadrage de la caméra de la carte : aucun effet sur la sécurité, les données ou l'infrastructure. La revue OWASP ne s'applique pas.
Taille : commit 1 : 135 ajouts et 25 suppressions. Commit 2 : 81 ajouts et 32 suppressions. Les deux restent sous 400 lignes.
Retour arrière
Un revert de la PR suffit : il n'y a ni migration ni donnée.
🤖 Generated with Claude Code