Comment 1 by Antoine Millet, Oct 7, 2009
Je pensais mettre en haut les champs vraiment obligatoires : - nom - description - url - durl Et en bas, affichés via JS avec soit un panneau déroulant, ou une coche, des options "avancées" vers tout le reste. Par contre j'ai pas vraiment le temps de me plonger dans JS et un de ses frameworks pour le moment, donc si tu veux bien t'en occuper... Pour ce qui est du lien "commandes avancées" dans commandes simples je ne suis pas contre...
Comment 2 by Jordan Bracco, Oct 7, 2009
L'URL de base (celle sans args) peut aussi être optionnelle. Genre dans 99% des cas, c'est l'host, et c'est facilement récupérable avec n'importe quelle bonne lib qui parse les URIs.
Comment 3 by Sunny Ripert, Oct 7, 2009
Ouais ça me plait le panneau déroulant d'options avancées ! +1 pour rendre durl optionnelle, je la verrais bien être juste la recherche vide par défaut
Comment 4 by Antoine Millet, Oct 7, 2009
Je suis ok pour la rendre optionnelle et prendre l'url de base dans ce cas. La recherche vide peut poser plus de problèmes sur certains sites, et puis c'est plus dans la logique de prendre l'url de base. Mais je suis contre la retirer totalement (c'est indispensable pour certains sites). Tu penses pouvoir t'occuper de ça (coté templates du moins, jpeux m'occuper du code si t'as vraiment pas envie de te plonger dedans) ?
Comment 5 by Sunny Ripert, Oct 7, 2009
Pour moi la recherche vide est plus logique (genre pour les commandes comme rapidshare) et renvoie vers la home pour certains autres (genre google). Je ne pense pas, non j'ai pas un week-end de libre avant le 21 novembre (en plus d'un week-end d'intégration j'héberge quelqu'un de nouveau chaque week-end d'ici là...) > Mais je suis contre la retirer totalement (c'est indispensable pour certains sites). Est-ce que c'est pas justement un truc qu'est possible de faire avec la nouvelle syntaxe ?
Comment 6 by Antoine Millet, Oct 7, 2009
"Je ne pense pas, non j'ai pas un week-end de libre avant le 21 novembre (en plus d'un week-end d'intégration j'héberge quelqu'un de nouveau chaque week-end d'ici là...)" -> Bon ben ça règle le problème, on verra ça plus tard.
Labels: Priority:Low, -Priority:Medium
Comment 7 by Antoine Millet, Oct 8, 2009
J'ajoute le ticket 18 qui concerne aussi le formulaire d'ajout : Sur e1 on avait sous chaque champ un exemple à rentrer, cela aidait à comprendre ce qu'il fallait mettre.
Summary: Refonte de l'ajout des commandes
Comment 8 by Benjamin Danon, Oct 18, 2009
Je suis aussi enthousiaste que Sunny pour le panneau déroulant avec les options avancées :) Et dans ce panneau, on pourrait aussi ajouté le code Lua ! Donc on aurait un seul formulaire. C'est plus simple je pense, ça ne force pas à choisir entre simple et avancé avant d'avoir rempli quoi que ce soit. Je trouve aussi que l'affichage des deux petits textes pour choisir complexifie encore plus l'ajout d'une commande :-° J'essaierai de proposer un formulaire concret bientôt ;)
Comment 9 by Antoine Millet, Oct 18, 2009
Mélanger commandes avancées et commandes normales je suis pas sûr que ce soit une super idée... La distinction est clairement faite partout sur le site entre les deux types de commandes, si on mélange tout, ça risque d'embrouiller les gens plutôt qu'autre chose. A la limite, une modification dynamique des champs du formulaire avec un choix radio simple/avancée et du JS me semblerait plus adapté. N'hésite pas à créer une branche pour faire ton formulaire, ça ne sera que plus facile à intégrer plus tard ;). (je t'ai donné les droits sur le repo git)
Sign in to reply to this comment.
Reported by Sunny Ripert, Oct 7, 2009