Skip to content

GPS: Améliorations d'ergonomie#5204

Merged
tonial merged 8 commits into
masterfrom
alaurent/gps
Dec 13, 2024
Merged

GPS: Améliorations d'ergonomie#5204
tonial merged 8 commits into
masterfrom
alaurent/gps

Conversation

@tonial

@tonial tonial commented Dec 6, 2024

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

capture 2024-12-09 à 15 12 41
capture 2024-12-09 à 15 46 20

@tonial tonial added the modifié Modifié dans le changelog. label Dec 6, 2024
@tonial tonial requested a review from hellodeloo December 6, 2024 09:03
@tonial tonial self-assigned this Dec 6, 2024
@tonial

tonial commented Dec 6, 2024

Copy link
Copy Markdown
Contributor Author

@hellodeloo je veux bien un coup de main pour le html de la box : je n'arrive pas à avoir le bouton "consulter la fiche" en une seule ligne tout en étant gérée par flex pour que sur un smartphone ça aille en dessous en toute largeur.

De plus, je remarque que l'image sur les cartes du dashboard sont tronquées (on perd le haut et le bas du dessin).
Je n'ai pas trouvé comment modifier pour que l'image soit affichée en toute hauteur et alignée à droite.

@tonial tonial force-pushed the alaurent/gps branch 4 times, most recently from c1650be to d3303f9 Compare December 12, 2024 11:54
@tonial

tonial commented Dec 12, 2024

Copy link
Copy Markdown
Contributor Author

Merci @hellodeloo !

@tonial tonial requested review from EwenKorr and leo-naeka December 12, 2024 11:57

@leo-naeka leo-naeka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tonial tonial added this pull request to the merge queue Dec 13, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Dec 13, 2024
@tonial tonial enabled auto-merge December 13, 2024 05:26
@tonial tonial added this pull request to the merge queue Dec 13, 2024
Merged via the queue into master with commit f58edd8 Dec 13, 2024
@tonial tonial deleted the alaurent/gps branch December 13, 2024 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants