Discussion MediaWiki:Gadget-CommonEdit.js

Remove redundant check

Hi!

I think we can change "forcePreview" code from

  if (wgUserName != null || wgAction != "edit") return;

to

  if (wgUserName != null) return;

because MediaWiki:Common.js/edit.js is only loaded when edit a page. (Or this would cause a problem with "|| wgAction == 'submit'" at MediaWiki:Common.js?) Heldergeovane (d) 20 juillet 2009 à 14:11 (CEST)

I'm not sure to understand: MediaWiki:Common.js/edit.js must be loaded both when the action is "edit" and "submit" (for instance the special caracters are loaded whenever during editing); but forcePreview must be loaded only the first time an editpage is loaded, else anonymous visitors couldn't never edit, isn't it? ~ Seb35 [^_^] 5 octobre 2009 à 19:40 (CEST)

Hello!

I was wondering if is it possible to check the value of wgAction only one time. Currently this is checked twice, at MediaWiki:Common.js:

if( wgAction == 'edit' || wgAction == 'submit' ) {
  importScript( 'MediaWiki:Common.js/edit.js' );
}

and at MediaWiki:Common.js/edit.js:

  if (wgUserName != null || wgAction != "edit") return;

But looking at the code now it seems not possible to do this simplification... What do you think? Heldergeovane (d) 17 octobre 2009 à 21:33 (CEST)

Barre Monobook

Trois choses :

• Ce serait bien de préciser qu'il ne s'agit pas d'une barre pour Monobook mais seulement d'une barre dans le style Monobook qu'il y avait auparavant.
• La barre "Vector" par défaut est personnalisable. Du coup, quel est l'intérêt de cette barre "Monobook" ?
• obtenir('MonobookToolbar') -> où est la source ?

— Dakdada (discuter) 3 mai 2013 à 14:02 (CEST)

• En effet.
• Elle est préférée par pas mal d'utilisateurs, et sa création par Mediawiki semble cassée, d'où la volonté de la créer d'ici à notre convenance. Cette barre "monobook style" aussi également personnalisable, de façon probablement plus simple, d'ailleurs.
• Mediawiki:Gadget-MonobookToolbar.js

⇨ D^r Brains ^{∞ Consultation ∞} 3 mai 2013 à 15:39 (CEST)

Merci. Sur le deuxième point, MonobookToolbar est un gadget donc, mais alors pourquoi le charger ici, pour tout le monde ? Il serait plus logique d'activer le gadget par défaut (désactivable si on veut). NB : Y a-t-il un lien qui discute de ce qui « semble cassé » ? — Dakdada (discuter) 3 mai 2013 à 16:03 (CEST)

Maybe MW:Talk:MediaWiki 1.22/wmf3. JackPotte ($♠) 4 mai 2013 à 13:08 (CEST)

@Darkdadaah : c'est ce qui a été fait, finalement. CF les bistros d'hier et d'aujourd'hui.

⇨ D^r Brains ^{∞ Consultation ∞} 4 mai 2013 à 13:57 (CEST)

Explication code dans initializeCharSubset()

Après avoir remarqué cette modif et la suivante, j'ai retrouvé la discussion en rapport : DIMS MediaWiki:Common.js/edit.js. od†n ↗^blah 31 décembre 2017 à 14:50 (CET)

Je viens de modifier le code de sorte à remplacer le javascript inline (href="javascript:...") par un event handler onclick, ce qui permet de résoudre tous ces problèmes d'escaping et d'injection javascript. od†n ↗^blah 31 décembre 2017 à 16:34 (CET)

Lien à garder sous le code

How to create memory leaks in jQuery

TL;DR: Les "event listeners" et "data" ajoutés via jQuery restent en mémoire indéfiniment si l'on supprime ensuite l'élément avec une méthode JavaScript natif (removeChild(), innerHTML = ''). Il faut supprimer l'élément avec la méthode jQuery : remove().

Le code est, actuellement, doublement protégé à ce niveau : aucun élément avec un event listener n'est ensuite supprimé, et toutes les suppressions se font en jQuery. Comme c'est le genre de bug qui peut apparaître dans le futur, je préfère quand même poster cette petite note.

TL;DR++: Supprimez les éléments avec jQuery.

od†n ↗^blah 18 mai 2018 à 20:12 (CEST)

Je précise aussi que, contrairement à ce qui était prévu dans l'article, la situation n'a pas changé avec jQuery 3. od†n ↗^blah 18 mai 2018 à 20:23 (CEST)