Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unités - Fiche unité sur la carte #1102

Closed
4 tasks done
ivangabriele opened this issue Jan 4, 2024 · 4 comments · Fixed by #1103, #1110 or #1133
Closed
4 tasks done

Unités - Fiche unité sur la carte #1102

ivangabriele opened this issue Jan 4, 2024 · 4 comments · Fixed by #1103, #1110 or #1133
Assignees
Labels
bug Something isn't working dev

Comments

@ivangabriele
Copy link
Contributor

ivangabriele commented Jan 4, 2024

Retours UX

Retours UI

  • Lorsque le commentaire sur le moyen est long, agrandir la carte du moyen pour afficher jusqu'à 3 lignes. Au-delà de 3 lignes, mettre des points de suspension et un survol.
    image

Originally posted by @AdelineCelier in MTES-MCT/monitorfish#2746 (comment)

@ivangabriele ivangabriele self-assigned this Jan 4, 2024
@ivangabriele ivangabriele added bug Something isn't working dev labels Jan 4, 2024
ivangabriele added a commit that referenced this issue Jan 4, 2024
## Related Pull Requests & Issues

- Resolve #1102

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

@ivangabriele les 3 premiers points sont bons pour moi, en revanche le point UI ne semble pas réglé :
image

@ivangabriele
Copy link
Contributor Author

Mais dans ce cas il faudrait que le fond gris à gauche s'adapte à la hauteur de la carte + qu'il y a plus de marge sous la dernière ligne

ivangabriele added a commit that referenced this issue Jan 8, 2024
## Related Pull Requests & Issues

- Resolve #1105
- Resolve #1102

----

- [ ] Tests E2E (Cypress)
@AdelineCelier
Copy link
Collaborator

@ivangabriele c'est mieux comme ça !
Bon et je vais faire la grosse reloue, mais le top serait que l'icône sur le fond gris se centre en hauteur, si c'est possible 😇
image

Bon idem pas bloquant pour la mep bien entendu !

@AdelineCelier
Copy link
Collaborator

c'est bon !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment