Enlever le bouton télécharger qui ne sert plus à rien #10
Conversation
javascript/bigup.js
Outdated
| $editer.attr('style',$editer.attr('data-prev-style')); | ||
| $editer.attr('data-prev-style',null); | ||
| $editer.addClass('editer_with_bigup').removeClass('biguping'); | ||
| $editer.parents('form').find('.boutons').first().remove(); |
There was a problem hiding this comment.
En fait, il faut hide() et pas remove(), sinon, ça casse le retour au formulaire quand on clique sur un des 3 autres mode et qu'on veut revenir au mode de téléversement.
En fait, il faut hide() et pas remove(), sinon, ça casse le retour au formulaire quand on clique sur un des 3 autres modes et qu'on veut revenir au mode de téléversement.
|
Je ne pense pas que ça doit être appliqué de façon générique sur la méthode .bigup(), mais plus, quitte à faire cela, sur celles spécifiques à un formulaire donné (par exemple dans bigup.logos.js). On n’est pas à l’abri qu’un formulaire quelconque possède plus de boutons que simplement "Téléverser". |
|
Je me suis effectivement posé la question. Donc, si tu me guides pour trouver formulaire par formulaire, je suis preneur, évidement, mais j'ai vraiment galéré ;-) |
|
en l'occurence il n'est pas rare de mettre un premier bouton caché pour qu'il prenne l'action par défaut quand on tape entrée dans le formulaire, et un tel patch le supprimerai. |
|
Nouveau patch, qui répond (je l'espère) aux objections : https://core.spip.net/issues/4446#note-11 |
|
Et voici le PR qui va bien : #13 |
3 participants
cf https://core.spip.net/issues/4446