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

Improve-feedback #207

Closed
wants to merge 7 commits into from
Closed

Improve-feedback #207

wants to merge 7 commits into from

Conversation

renardpal
Copy link
Contributor

@renardpal renardpal commented May 14, 2019

Fixes #204

j'ai oublié les alt sur les img (yes / kindof / not really" ! et le js marche pas, source ici :
https://codepen.io/smcllns/pen/GrXVrE

@renardpal renardpal requested a review from sandcha May 14, 2019 10:01
@sandcha
Copy link
Contributor

sandcha commented May 14, 2019

À faire : exécuter une fonction selon le clic.

  • Si vert, merci.
  • Si orange ou rouge, dites en nous plus et mailto.

@renardpal
Copy link
Contributor Author

still to be done:

  • make the form to actually send emails !
  • add a "selected" effect for smileys

@benjello
Copy link
Member

benjello commented Feb 6, 2020

@sandcha @maukoquiroga : je passe par là et découvre cette PR orpheline. On en fait quoi ?

@renardpal
Copy link
Contributor Author

@sandcha @maukoquiroga @benjello je ne sais pas s'il est judicieux de garder cette fonctionnalité si personne ne peux faire de chantier d'amélioration de la doc derrière... c'est une PR d'un autre temps !

@benjello
Copy link
Member

Merci @renardpal ! @sandcha @maukoquiroga : your call !

@MattiSG
Copy link
Member

MattiSG commented Apr 29, 2022

Avec les données fournies par la première version et indiquées dans #261, je pense qu'il vaut mieux fermer 🙂

@MattiSG MattiSG closed this Apr 29, 2022
@MattiSG MattiSG deleted the improve-feedback branch April 29, 2022 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add feedback section
5 participants