Produits Entr'ouvert » Calebasse

Juste pour la cohérence, si on utilise .on(), je serai pour l'utiliser systématiquement et ne plus utiliser toutes les méthodes directes de mise en place de handler comme .click() et .change(), en plus à relire le .on() est plus explicite alors que .change() ou .click() peuvent être lu comme des verbes, et sans argument il me semble aussi que ça provoque un .trigger() (mais je n'en suis plus sûr) ce que .on() ne fait pas.

Quand à l'utilisation de beforeunload, apparemment renvoyer une chaîne ne marche que sur webkit: https://developer.mozilla.org/fr/docs/DOM/window.onbeforeunload (ils disent Safari mais pour moi ça veut dire webkit)

Sur IE et Firefox il faudrait faire e.returnValue = '...'.

Idéalement il faudrait comme pour les actes vérifier que ce qui a vraiment changé, à partir de jQuery 1.6 il est possible de récupérer cette valeur initiale avec $e.attr('value') ou $e.attr('checked') != undefined pour les checkbox et boutons radios.

Benjamin Dauvergne a écrit :

Juste pour la cohérence, si on utilise .on(), je serai pour l'utiliser systématiquement et ne plus utiliser toutes les méthodes directes de mise en place de handler comme .click() et .change(), en plus à relire le .on() est plus explicite alors que .change() ou .click() peuvent être lu comme des verbes, et sans argument il me semble aussi que ça provoque un .trigger() (mais je n'en suis plus sûr) ce que .on() ne fait pas.

C'est modifié.

Quand à l'utilisation de beforeunload, apparemment renvoyer une chaîne ne marche que sur webkit: https://developer.mozilla.org/fr/docs/DOM/window.onbeforeunload (ils disent Safari mais pour moi ça veut dire webkit)

Sur IE et Firefox il faudrait faire e.returnValue = '...'.

Sous Firefox ça semble beaucoup plus tordu que ça : https://bugzilla.mozilla.org/show_bug.cgi?id=588292
J'ai essayé le coup du returnValue mais le comportement reste le même. En gros Firefox n'en a rien a foutre du beforeunload il gère ça à sa sauce et ça me convient dans le cas présent.
Pour IE je n'ai pas testé.
Tous les exemples que j'ai trouvé sur jquery avec un beforeunload ne font que renvoyer une string. Pour le moment je laisse comme ça.

Idéalement il faudrait comme pour les actes vérifier que ce qui a vraiment changé, à partir de jQuery 1.6 il est possible de récupérer cette valeur initiale avec $e.attr('value') ou $e.attr('checked') != undefined pour les checkbox et boutons radios.

C'est un peu lourd à gérer pour une plus value qui me semble faible. Dans l'immédiat je pense qu'on peut s'en passer.

Appliqué par commit calebasse|commit:83fdead5e5dde28acf6ea0e75519632e1f6478f6.