Projet

Général

Profil

Development #22438

Utiliser <button> plutôt que <input>

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
10 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Ça offre comme gros avantage ensuite au niveau des styles d'avoir accès aux sélecteurs ::before et ::after, pour par exemple intégrer une icône, une flèche, dans le bouton.


Fichiers

Révisions associées

Révision 85545e5d (diff)
Ajouté par Frédéric Péters il y a environ 6 ans

misc: switch form buttons to <button> markup (#22438)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

C'est à dire partout ? Le problème que j'avais avec button dans le temps c'est que sous IE (mais certainement il y a longtemp) l'attribut name n'était pas pris en compte, si ce problème n'en est plus un, sûr changeons cela partout.

#2

Mis à jour par Frédéric Péters il y a environ 6 ans

Via #14467 la modif a eu lieu dans w.c.s. et est en prod depuis plus d'un an; par rapport à IE j'y notais « Les derniers bugs qui restaient dans la gestion de <button> c'était MSIE7 ».

#3

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Ok pour moi sur le principe.

#4

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Assigné à mis à Frédéric Péters
#5

Mis à jour par Frédéric Péters il y a environ 6 ans

À noter qu'il reste des templates dans idp_openid auxquels je ne touche pas (ils contiennent des chaines en anglais donc je me dis qu'on ne les voit vraiment nulle part).

Il y a ça côté js :

-        $(document).on('click.manager', 'table tr[data-url][rel=popup], button[rel=popup]', displayPopup);
+        $(document).on('click.manager', 'table tr[data-url][rel=popup]', displayPopup);

Parce qu'il se fait déjà maintenant attraper un peu plus haut dans le changement :

-        $(document).on('click', 'input[type=submit]', function (e) {
+        $(document).on('click', 'button', function (e) {

Comme modification dans le rendu il y a le bouton "Modifier" (les rôles) de la page d'un utilisateur qui désormais prend toute la largeur, comme les autres actions, je peux le réduire si souhaité. (voire le remplacer par un simple lien)

#6

Mis à jour par Frédéric Péters il y a environ 6 ans

Comme modification dans le rendu il y a le bouton "Modifier" (les rôles) de la page d'un utilisateur qui désormais prend toute la largeur, comme les autres actions, je peux le réduire si souhaité. (voire le remplacer par un simple lien)

Voilà un patch qui ne change pas la taille de ce bouton. (si jamais ça avait une importance)

#7

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Toujours ack.

#8

Mis à jour par Frédéric Péters il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 85545e5d3550df95d72bf185c1436edd0a6c84db
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Mar 21 17:51:12 2018 +0100

    misc: switch form buttons to <button> markup (#22438)
#9

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF