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

[WIP] Utilise un datepicker HTML5 pour le sélecteur de date des Unes #4110

Closed
wants to merge 1 commit into from
Closed

Conversation

Eskimon
Copy link
Contributor

@Eskimon Eskimon commented Jan 3, 2017

Q R
Type de modification correction de bug
Ticket concerné #3605

QA

  • Essayer de créer une Une
  • Vérifier que le champ de sélection de la date est bien un sélecteur de date au lieu d'un simple champ texte
  • Idem sur mobile
  • Idem² pour l'édition d'une Une

@Eskimon Eskimon changed the title featured content date selector is now an html5 datepicker Utilise un datepicker HTML5 pour le sélecteur de date des Unes Jan 3, 2017
@pierre-24
Copy link
Member

Oh, une PR de @Eskimon <3

@Eskimon
Copy link
Contributor Author

Eskimon commented Jan 3, 2017

keur

PS, oui j'avais bien vu la PR d'Emeric, mais comme je n'ai pas le temps de me concentrer sur des grandes périodes de temps je préfère vous proposer un petit fix qu'un grand refacto (et tant pis, procédons par étape, au moins la com' aura déjà ça !)

@coveralls
Copy link

coveralls commented Jan 3, 2017

Coverage Status

Coverage remained the same at 87.103% when pulling 829c927 on Eskimon:fix-3605 into bd756c7 on zestedesavoir:dev.

@pierre-24
Copy link
Member

Rapport de QA: quasiment OK, sauf que si on met une date, puis qu'on essaye de la changer, le datepicker ne propose pas la date précédente mais "dd/mm/yyyy".

@pierre-24 pierre-24 added C-Front Concerne l'interface du site QA svp labels Jan 10, 2017
@artragis
Copy link
Member

artragis commented Feb 2, 2017

j'ai tendance à penser que le bug que tu retournes @pierre-24 n'est pas vraiment important, ni même gênant. @vhf tu en penses quoi? on merge?

@Eskimon
Copy link
Contributor Author

Eskimon commented Feb 7, 2017

Faudrait que je test pour reproduire et bien comprendre mais dans les faits je me sert uniquement du composant natif au navigateur, donc pas sur que je puisse faire quoi que ce soit de toute facon.

@Situphen
Copy link
Member

Actuellement la valeur attendue est 2016-12-25 00:00:00, en mettant type="date" les navigateurs compatibles vont renvoyer une date sous la forme 2016-12-25 ce qui n'est pas ce que l'on veut (il manque l'heure). Il existe type="datetime-local" qui renvoit une date et une heure sous la forme 2016-12-25T00:00:00, il y a probablement moyen de se débrouiller pour enlever/rajouter le T au bon moment entre le front et la base de donnée je suppose ? :)

@artragis
Copy link
Member

@Eskimon tu es toujours là?

@Eskimon
Copy link
Contributor Author

Eskimon commented Apr 10, 2017

Oui mais pas tellement le temps de reprendre quoi que ce soit... :S

@vhf vhf changed the title Utilise un datepicker HTML5 pour le sélecteur de date des Unes [WIP] Utilise un datepicker HTML5 pour le sélecteur de date des Unes Aug 5, 2017
@pierre-24
Copy link
Member

En fait, c'est telement idiot que je vais mettre ça dans #4450 :)

@pierre-24 pierre-24 closed this Aug 9, 2017
@gllmc gllmc removed the QA svp label Aug 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants