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

Change le type de champ pour la date de publication des unes #6544

Merged
merged 2 commits into from
Dec 9, 2023

Conversation

philippemilink
Copy link
Member

Fix #6524

Contrôle qualité

  • Se connecter en tant qu'admin
  • Créer une nouvelle une (par exemple en allant sur un contenu publié et cliquer sur Ajouter en une
  • Le navigateur propose un datepicker pour le champ Date de publication (si le navigateur le supporte)
  • Valider le formulaire : tout fonctionne.

@philippemilink philippemilink added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 14, 2023
@coveralls
Copy link

coveralls commented Oct 14, 2023

Coverage Status

coverage: 88.81%. remained the same
when pulling 63a6eee on philippemilink:date-field-featured
into c15b9bf on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

En plus de ce que tu as fait, j'ai quelques remarques et questions :

  • je mettrai à jour le label, pour faire disparaître la mention du format avec le T dans les exemples
  • voir si les input_formats sont toujours nécessaires ?
  • le placeholder n'est pas affiché chez moi, donc peut-être l'enlever aussi ?
  • est-ce que le format pour le widget est nécessaire si on a déjà un datetime-local ?

@philippemilink
Copy link
Member Author

  • je mettrai à jour le label, pour faire disparaître la mention du format avec le T dans les exemples

C'est fait.

  • voir si les input_formats sont toujours nécessaires ?

  • le placeholder n'est pas affiché chez moi, donc peut-être l'enlever aussi ?

  • est-ce que le format pour le widget est nécessaire si on a déjà un datetime-local ?

J'y ai pensé, mais comme c'est dit sur la doc de Mozilla : "Les navigateurs qui n'implémentent pas cette fonctionnalité afficheront un contrôle de saisie textuelle.". Donc si jamais ce n'est pas implémenté, c'est comme si on avait un <input type="text">, donc autant tout garder comme si c'était du texte.

Bon, après, tu pourras me rétorquer qu'en fait c'est déjà supporté partout...

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Oct 23, 2023

Pour moi, c'est supporté par toutes les versions récentes, et en particulier celles utilisées par l'équipe avec les droits pour cette page. Après si tu veux le laisser ou pas, je te laisse décider, ça ne change pas la face du monde. ^^ Les deux me vont.

@philippemilink
Copy link
Member Author

C'est vrai que c'est sur une page qui n'est consultée que par une poignée de membres... J'ai enlevé tout ce qui n'était plus nécessaire.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) December 9, 2023 18:01
@Arnaud-D Arnaud-D merged commit 77de7ca into zestedesavoir:dev Dec 9, 2023
8 checks passed
@philippemilink philippemilink deleted the date-field-featured branch December 9, 2023 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Faciliter la programmation des Unes en modifiant le type de la date de publication.
3 participants