Superdrac/spawnkill — pull request #1 by dorian-marchal (2014-08-15 09:31:11+00:00)
Title: Modifications du plugin Raccourcis

Salut,

Merci encore pour ton module, je suis en train de le mettre en prod.

J'ai fait quelques modifs pour que ton code soit dans les conventions que je me suis donné :
- Les modules sont en anglais : Renommage de `Raccourcis` en `Shortcuts`
- Les fonctions sont en anglais : Renommage des fonctions en `previousPage` et `nextPage`
- Toutes les chaînes de caractères sont wrappées dans des `"` et non des `'`
- Tous les opérateurs doivent avoir un espace avant et un espace après : ` = `, ` + `, ` < `, ` +=`, ...
- Les blocs sont formatés comme ça (l'accolade juste après la parenthèse avec un espace avant) :
```
if (var === "test") {
doSomething();
}
```

Mise à part ça, j'ai vu que dans la fonction `nextPage`, tu allais chercher plusieurs fois les données dans le DOM.
J'ai pas fait la correction, mais pour optimiser le truc, il faudrait faire quelque chose comme ça :
```javascript
var $paginationLinks = $(".pagination").eq(0).find("a");

//Permet de récupérer le nombre de page sur les gros topics
if($paginationLinks.last().html() === "»") {
nbPageMax = parseInt($paginationLinks.eq($paginationLinks.length - 2).html());
}
else {
nbPageMax = parseInt($paginationLinks.last().html());
}
```

Voilà, merci encore à toi pour ta contribution, c'est cool :smile: