Update playing.js #55
Conversation
Ceci est une démonstration de l'ajout du formatage des planètes de l'Astro dans la "ToolTip" avec quelques modifications supplémentaire : - Les zones mortelles se trouvent différenciées par la graisse, l'italique et la couleur liée à cette dernière ; suivies de l'icône Tête de mort de Mush ; - Les zones à risque Mush (Crystalite) se retrouve différenciée par l'italique de couleur suivie de l'icône Mush ; - Les zones d'intérêt (O2 et Fuel) sont différenciées par la graisse suivies de l'icône O2 ou Fuel correspondantes. Images illustrant le propos. Par manque de case nécessaire au test, l'Océan était similaire au Fuel et le Marais à un Mankarog pour l'exemple. http://imgup.motion-twin.com/twinoid/d/0/0b62b13b_6075278.jpg http://imgup.motion-twin.com/twinoid/3/3/e3819f4c_6075278.jpg Qu'en pensez-vous ?
Changement pour le nom de la case en **Sismique**
| var caseNameRecup = casenamereg.exec($(this).attr("onmouseover"))[1]; | ||
| var finalCaseName = '',finalCaseNameToolTip = ''; | ||
| switch(caseNameRecup){ | ||
| case 'Sismique': |
There was a problem hiding this comment.
Le texte est différent selon la langue du jeu donc faudrait passer le texte FR dans la fonction Main.k.text.gettext()
There was a problem hiding this comment.
Tout à fait d'accord. N'ayant de compte sur les deux autres Mush, je ne m'étais attardé sur la question. De plus, le nom de la section/zone/case est différent de celui relevé dans les Wikis (c'est le cas pour l'activité sismique et celle volcanique). Comme je fais une petite pause sur le jeu en ce moment, je repousse la modification cette partie.
There was a problem hiding this comment.
Si tu ne joue pas sur les autres versions, juste utiliser Main.k.text.gettext suffit. A un joueur d'une autre langue de passer ici pour traduire.
| if (dist && dir) tooltipret += "<em>" + dir + " - " + dist + " <img alt=\"Icône Fuel\" src=\"http://data.twinoid.com/proxy/mush.vg/img/icons/ui/fuel.png\"/></em><br />"; | ||
| ret += '---\n'+cases.join(", "); | ||
| tooltipret += '--<br />'+casesToolTip.join(", "); | ||
| if(nbCasesInconnues === 1){ |
There was a problem hiding this comment.
Tu peux me donner un exemple de ce que ça donne quand case.length === 0 ? j'ai du mal à visualiser.
Tu as oublié Main.k.text.gettext ;)
Je trouve aussi que ça fait beaucoup de if et chaque texte est copié/collé 4 fois. C'est pas tip top.
Tu peux utiliser Main.k.text.ngettext pour gérer les pluriels facilement. Voir un exemple ici :
ctrl-w/src/main/tabs/playing.js
Line 3448 in a274302
There was a problem hiding this comment.
Ah ! Nickel que cette fonction existe, merci ^^
Pour le case.length === 0, ah vrai dire, je ne sais pas si cette situation peut apparaitre : une planète venant d'être tout juste découverte semble avoir toujours une section ou plus (inconnue). Alors je n'ai jamais réfléchi à l'éventualité qu'aucune section soit présente. Je vais creuser la question.
| Main.k.text.gettext("Insère la liste de planètes dans la zone de texte active, de la forme :</p><p>" + | ||
| "TODO: aperçu") | ||
| Main.k.text.gettext("Insère la liste de planètes dans la zone de texte active, de la forme :<br />" + | ||
| toolTipText + "</p>") |
There was a problem hiding this comment.
C'est utile ce <p> à la fin ? Il y était de base mais c'est du vieux code. Je crois que je ne suis pas repassé dessus. Si on peut en profiter pour nettoyer les trucs inutiles je suis preneur :).
There was a problem hiding this comment.
Étourdis comme je le suis, et vous l'avez constaté il y a peu, je ne risque pas d'être d'une grande aide. Mais je me ferais un plaisir de débusquer au filet à papillon les inscriptions caduques ^^
There was a problem hiding this comment.
^^ Faudrait juste tester le jeu en l'enlevant voir si ça change quelque chose, si c'est pas le cas, on vire ! :)
|
Bonne idée, depuis le temps que ça traîne cette TODO. J'ai fait une review vite fait. Faut pas prendre peur, ya plein de commentaires mais c'est juste des petits soucis. |
Co-Authored-By: badconker <badconker@users.noreply.github.com>
Co-Authored-By: badconker <badconker@users.noreply.github.com>
Co-Authored-By: badconker <badconker@users.noreply.github.com>
Co-Authored-By: badconker <badconker@users.noreply.github.com>
Co-Authored-By: badconker <badconker@users.noreply.github.com>
|
Vous êtes tout pardonné, tout comme je l'espère l'être pour les errements dont je suis l'auteur ^^ |
Depuis avant que tu reprennes le script d'ailleurs, non ? xD |
|
@kLabz oui tout à fait ;) |
Ceci est une démonstration de l'ajout du formatage des planètes de l'Astro dans la "ToolTip" avec quelques modifications supplémentaire :
Images illustrant le propos. Par manque de case nécessaire au test, l'Océan était similaire au Fuel et le Marais à un Mankarog pour l'exemple.
Qu'en pensez-vous ?