-
Notifications
You must be signed in to change notification settings - Fork 67
Nouvelle interface de commande de billets #1793
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
base: master
Are you sure you want to change the base?
Conversation
ça rends très bien ce nouveau tunnel ! (notamment le choix du moyen du paiement) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques petites remarques.
Le rendu est beaucoup mieux que le tunnel actuel !
Et il faut qu'on ait une conversation à long terme sur le code front. Je me note d'en parler au point outils.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour tout ! C'est vraiment top.
Juste du pinaillage, pour moi je valides !
|
||
<!-- Steps Indicator --> | ||
<div class="steps-indicator" data-current-step="1"> | ||
<div class="step-item active" data-step="1"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ce serait possible de rendre clickable ces "onglets" ?
Si c'est trop relou, on voit ça dans une autre PR
|
||
{{ form_start(ticketForm, {attr: {id: 'formulaire'}}) }} | ||
{{ form_start(ticketForm, {attr: {id: 'formulaire', class: 'ticket-container'}}) }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est possible d'ajouter la mention "* indique un champ obligatoire" ?
{% endfor %} | ||
|
||
<div class="ticket-actions"> | ||
<button type="button" class="btn btn-outline btn-prev"><i class="fas fa-arrow-left"></i> Précédent</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faudrait une marge à droite de l'icone pour ce bouton précédent.
Voici une proposition de refonte pour le parcours de commande de billets.
Rendu desktop: https://youtu.be/S0Dfiv-jL3E
Rendu mobile: https://youtube.com/shorts/2VUs3qGZUgk