Affichage des ids des personnages recontrés, Wanted et BL

Tout ce qui concerne le site web et les extensions du jeu pour Firefox et Chrome
Répondre
Avatar de l’utilisateur
Carl Thurner
Messages : 1333
Inscription : 27 mai 2007 17:23
Localisation : Vancouver, BC, Canada

Affichage des ids des personnages recontrés, Wanted et BL

Message par Carl Thurner » 31 mai 2015 09:00

Bonjour John,

Si jamais tu passes par là, j'ai pas mal cherché, et cette extension est responsable des souci d'affichage des wanted et de la liste noire sur chrome sous Windows (les wanted s'affichent en plein écran au lieu de d'avoir le petit popup attendu).
La partie en "cause" est l'otpion "afficher l'id des persos dans la fiche de rencontre".

Ma propositioin de code est la suivante (ligne 3707 de ton LibertyIsland_BrowserOverlay.js) :

Code : Tout sélectionner

// Ajout pour empêcher que l'id n'apparaisse deux fois dans la fiche de rencontre (arrive parfois)
if (document.getElementById("LIE_" + myself.id) === null) {
	// Ajout de gras autour de IdPerso (difficile d'y croire mais ça corrige le souci d'affichage des wanted)
	// J'ai aussi enlevé les sous-lignage autour de Id pour que ça ressemble au reste des informations sur la fiche de rencontre
	nodeTableCellAvatar.nextSibling.nextSibling.innerHTML = contenuActuel + '<br><span id="LIE_' + myself.id + '">Id</span> : <b>' + idPerso + '</b>';
}
Qui replacerait :

Code : Tout sélectionner

nodeTableCellAvatar.nextSibling.nextSibling.innerHTML = contenuActuel + '<br><span id="LIE_' + myself.id + '"><u>Id</u></span> : ' + idPerso;
Good night, and good luck !

Avatar de l’utilisateur
Ely
Messages : 11560
Inscription : 27 févr. 2007 22:03
Localisation : Liège
Contact :

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par Ely » 31 mai 2015 23:29

je lui ai envoyé un email pour essayer de lui faire un mémo
http://www.pirates-caraibes.com
Jeu de rôle en ligne entièrement gratuit (MMORPG)

Avatar de l’utilisateur
John Ch 'le borgne' R
Site Admin
Messages : 791
Inscription : 18 nov. 2006 10:54
Localisation : chambre d'ami de Titus Pullo
Contact :

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par John Ch 'le borgne' R » 01 juin 2015 09:02

je jette un oeil..
la liste noire... ceux que tu ajoutes dans les paramètres de l'extension ?


bon je viens de faire un affichage de la fiche de rencontre de Clodomir, je ne vois rien qui cloche
ça fiche est bien affichée et je vois l'icône de wanted, je passe dessus, j'ai bien la petite fenêtre qui s'affiche (bon parfois ç bug et ça ne s'affiche plus ensuite...)

après une vingtaine de clic sur l'icône wanted, j'ai réussi à voir le problème

Avatar de l’utilisateur
Carl Thurner
Messages : 1333
Inscription : 27 mai 2007 17:23
Localisation : Vancouver, BC, Canada

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par Carl Thurner » 01 juin 2015 09:24

"la liste noire... ceux que tu ajoutes dans les paramètres de l'extension ?"

Non je parle de la liste noire et des wanted gérés par le jeu lui-même ;)



Ps : typiquement si je clique ici
Image

Cela m'ouvre la black liste en jeu en plein écran, mais pas si l'extension est désactivée (et plus précisément l'option sur l'affichage des ids des autres joueurs).
Good night, and good luck !

Avatar de l’utilisateur
Carl Thurner
Messages : 1333
Inscription : 27 mai 2007 17:23
Localisation : Vancouver, BC, Canada

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par Carl Thurner » 01 juin 2015 14:32

Autant pour moi : la modification proposée ne corrige pas complètement le souci.
J'essayer de chercher à nouveau ce soir.
Good night, and good luck !

Avatar de l’utilisateur
John Ch 'le borgne' R
Site Admin
Messages : 791
Inscription : 18 nov. 2006 10:54
Localisation : chambre d'ami de Titus Pullo
Contact :

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par John Ch 'le borgne' R » 01 juin 2015 23:50

c'est bon, je viens de jeter un oeil
ce n'est pas ce qui est injecté qui pose problème mais la méthode

voilà ce qu'il faut faire...
même si le innerHTML ne passe pas la vérif firefox, je devrais remplacer tout ça par des createNode et createTextNode à chaque fois

je ferai une vérin globale avec de poster une nouvelle version...

Code : Tout sélectionner

					/*
					 * on affiche son id si configurer dans les options du xpi
					 */
					if (response.values.ShowOthersId === "true")
					{
						var brForId = document.createElement("br");
						var spanForId = document.createElement("span");
						spanForId.id = 'LIE_' + LibertyIsland.escapeHTML(myself.id);
						spanForId.innerHTML = '<u>Id</u> : ' + LibertyIsland.escapeHTML(idPerso);
						nodeTableCellAvatar.nextSibling.nextSibling.appendChild(brForId);
						nodeTableCellAvatar.nextSibling.nextSibling.appendChild(spanForId);
						//nodeTableCellAvatar.nextSibling.nextSibling.innerHTML = contenuActuel + '<br><span id="LIE_' + LibertyIsland.escapeHTML(myself.id) + '"><u>Id</u></span> : ' + LibertyIsland.escapeHTML(idPerso);
					}

en propre...

Code : Tout sélectionner

					if (response.values.ShowOthersId === "true")
					{
						var brForId = document.createElement("br");
						var spanForId = document.createElement("span");
						spanForId.setAttribute('id', 'LIE_' + LibertyIsland.escapeHTML(myself.id));
						var uForId = document.createElement('u');
						var txtNodeForU = document.createTextNode('Son id');
						uForId.appendChild(txtNodeForU);
						spanForId.appendChild(uForId);
						var txtNodeForId = document.createTextNode(' : ' + LibertyIsland.escapeHTML(idPerso));
						spanForId.appendChild(txtNodeForId);
						nodeTableCellAvatar.nextSibling.nextSibling.appendChild(brForId);
						nodeTableCellAvatar.nextSibling.nextSibling.appendChild(spanForId);
					}

Avatar de l’utilisateur
Carl Thurner
Messages : 1333
Inscription : 27 mai 2007 17:23
Localisation : Vancouver, BC, Canada

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par Carl Thurner » 02 juin 2015 09:16

Bien vu !
Good night, and good luck !

Avatar de l’utilisateur
John Ch 'le borgne' R
Site Admin
Messages : 791
Inscription : 18 nov. 2006 10:54
Localisation : chambre d'ami de Titus Pullo
Contact :

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par John Ch 'le borgne' R » 06 juin 2015 20:24

bon j'ai tout réécrit les injecteurs et listeners...
je n'ai plus détecté de problème
innerHTML ne sert plus que pour reprendre les infos dans une chaine et faire les détections, tout ce qui est injection ne marche plus qu'avec createElement

version de l'extension pour chrome 0.2.1

je dois encore tout réécrire pour firefox...

Avatar de l’utilisateur
Carl Thurner
Messages : 1333
Inscription : 27 mai 2007 17:23
Localisation : Vancouver, BC, Canada

Re: Affichage des ids des personnages recontrés, Wanted et BL

Message par Carl Thurner » 06 juin 2015 23:22

Joli boulot.
Bon courage ;)
Good night, and good luck !

Répondre