Skip to content

Fix le soft-lock de la carte après le clic sur un signalement qui overlap un bâtiment#383

Closed
leonkenneth wants to merge 2 commits intomainfrom
fix-unable-to-drag-after-selecting-report-overlapping-building
Closed

Fix le soft-lock de la carte après le clic sur un signalement qui overlap un bâtiment#383
leonkenneth wants to merge 2 commits intomainfrom
fix-unable-to-drag-after-selecting-report-overlapping-building

Conversation

@leonkenneth
Copy link
Contributor

@leonkenneth leonkenneth commented Dec 3, 2025

Reproduction steps:

  • Se connecter
  • Aller sur le lien cible ci-dessus
  • Cliquer sur le signalement sans bouger
  • Essayer maintenant de se déplacer sur la carte (KO dans le premier cas, OK dans le second)

Le fix est dans le plugin mapbox-gl-draw : https://github.com/mapbox/mapbox-gl-draw/compare/main...leonkenneth:mapbox-gl-draw:fix-unable-to-drag-after-selecting-overlapping-features?expand=1

Reste à faire côté mapbox-gl-draw :

  • Analyse plus poussée qui valide que c'est bien onStop qui stop les events handlers ✅
  • Rajouter des tests
  • Ouvrir une PR sur le repo target

@vercel
Copy link

vercel bot commented Dec 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
batid-site Ready Ready Preview Comment Dec 8, 2025 1:55pm

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

Résultats des tests E2E Playwright

failed  17 failed
passed  1 passed
skipped  6 skipped

Details

stats  24 tests across 4 suites
duration  9 minutes, 53 seconds
commit  627ab89

Failed tests

chromium › definition-page.spec.ts › Définition & Standard › doit contenir les éléments nécessaires
firefox › definition-page.spec.ts › Définition & Standard › doit contenir les éléments nécessaires
chromium › edition-page.spec.ts › Edition › Désactivation › doit pouvoir désactiver un bâtiment
chromium › edition-page.spec.ts › Edition › Scission › doit pouvoir scinder un bâtiment
chromium › edition-page.spec.ts › Edition › Scission › doit pouvoir afficher une erreur de validation sur un des enfants
chromium › home-page.spec.ts › Page d'accueil › doit contenir les éléments nécessaires
chromium › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'ID-RNB pour la carte
chromium › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'adresse pour la carte qui fonctionne avec clavier
chromium › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'adresse pour la carte qui fonctionne avec la souris
chromium › home-page.spec.ts › Page d'accueil › empêche l'execution d'un script inline (CSP)
firefox › home-page.spec.ts › Page d'accueil › doit contenir les éléments nécessaires
firefox › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'ID-RNB pour la carte
firefox › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'adresse pour la carte qui fonctionne avec clavier
firefox › home-page.spec.ts › Page d'accueil › doit contenir le champ de recherche rapide d'adresse pour la carte qui fonctionne avec la souris
firefox › home-page.spec.ts › Page d'accueil › empêche l'execution d'un script inline (CSP)
chromium › map-page.spec.ts › Carte › doit contenir les éléments nécessaires
chromium › map-page.spec.ts › Carte › doit pouvoir afficher les données d'un bâtiment

Skipped tests

firefox › definition-page.spec.ts › Définition & Standard › ne doit pas avoir de lien d'image cassé
firefox › edition-page.spec.ts › Edition › Désactivation › doit pouvoir désactiver un bâtiment
firefox › edition-page.spec.ts › Edition › Scission › doit pouvoir scinder un bâtiment
firefox › edition-page.spec.ts › Edition › Scission › doit pouvoir afficher une erreur de validation sur un des enfants
firefox › map-page.spec.ts › Carte › doit contenir les éléments nécessaires
firefox › map-page.spec.ts › Carte › doit pouvoir afficher les données d'un bâtiment

@leonkenneth leonkenneth changed the title Fix unable to drag after selecting report overlapping building Fix le soft-lock de la carte après le clic sur un signalement qui overlap un bâtiment Dec 3, 2025
@leonkenneth leonkenneth marked this pull request as ready for review December 3, 2025 14:28
@leonkenneth leonkenneth marked this pull request as draft December 8, 2025 09:35
@leonkenneth leonkenneth force-pushed the fix-unable-to-drag-after-selecting-report-overlapping-building branch from e47eae7 to 627ab89 Compare December 8, 2025 13:53
@leonkenneth leonkenneth marked this pull request as ready for review December 8, 2025 14:24
@leonkenneth
Copy link
Contributor Author

Ouvert mapbox/mapbox-gl-draw#1491.

@leonkenneth leonkenneth closed this Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant