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

Mon feedback #1

Closed
pixeline opened this issue Aug 27, 2017 · 0 comments
Closed

Mon feedback #1

pixeline opened this issue Aug 27, 2017 · 0 comments

Comments

@pixeline
Copy link

  • UX: de très bonnes idées, attention juste au jaune clair sur bleu clair, pas assez contrasté. Mais le choix des sliders = top ("fuzzy logic") plutôt que de devoir choisir entre 1 à 7, c bien vu.
  • la validation devrait vérifier que toutes les questions sont remplies.
  • readme très utile, bien vu
  • organisation des fichiers: place les fichiers statiques (dossier "assets") dans leur propre dossier, ainsi que les fichiers partiels (ceux qu'on inclut, en php).
  • attention à ne pas révéler ton mdp dans ton git
    $bdd = new PDO('mysql:host=localhost;dbname=nomophobia;charset=utf8', 'root', 'root');
  • mysql: tu utilises une requête préparée pour l'insert, très bien. fais en de même pour le Select
    $postCode = $_GET["postCodeKey"];
  • c'est un peu étrange d'avoir tes questions dans un fichier json, alors que tu utilises une database. Perso, j'aurais tout mis dans la database.
  • dommage que tu ne respectes pas l'amélioration progressive: ton formulaire ne fonctionne pas sans javascript.
@ddvdv ddvdv closed this as completed Aug 29, 2017
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

No branches or pull requests

2 participants