georchestra / cadastrapp

Cadastre application for geOrchestra
GNU General Public License v3.0
10 stars 19 forks source link

Problème affichage fiche parcelle #216

Closed jusabatier closed 7 years ago

jusabatier commented 8 years ago

J'ai encore et toujours le même problème qui revient chez certains de mes utilisateurs (aléatoirement).

Voici un exemple de celui-ci :

image

Donc l'utilisateur essaye d'ouvrir le fiche de la parcelle, mais rien ne s'affiche.

Cependant les données sont bien présentes dans les requêtes réseau, et ajoutées dans le DOM html.

J'ai constaté que le problème semble venir de Ext, et plus particulièrement du <form> sélectionné sur mon screenshot dans l'outil dev de firefox, en effet sa propriété "height" est fixée à 2px, ce qui empêche la fiche de la parcelle de s'afficher (idem pour n'importe quel autre onglet).

Ce problème est très handicapant chez nous...

Donc si quelqu'un a une idée pour le résoudre ?

Personne d'autre n'a constaté ce comportement chez certains de ses utilisateurs ?

landryb commented 8 years ago

Et la console dit quoi ?

jusabatier commented 8 years ago

Ben rien comme erreur dans la console justement c'est bien ça qui est bizarre...

Aucuns logs / warning / erreur

MaelREBOUX commented 8 years ago

Je n'ai pas constaté ce pb ici.

jusabatier commented 8 years ago

J'ai trouvé de nouveaux logs concernant ce problème, ceux-cis arrivent après avoir cliqué sur rechercher, et lors de l'utilisation de "recentrer sur la selection" :

14:54:40,581 loadParcelleStore : [object Object]430157AN searchUtils.js:98:5
14:54:45,599 loadParcelleStore : [object Object]430157AP searchUtils.js:98:5
14:54:47,830 Not enough data to call the webservice  searchParcelleByRef.js:486:37
14:54:52,150 TypeError: features[0].geometry.bounds is null

Du coup l'affichage de la fiche de parcelle est impossible, ainsi que le recentrage sur celle-ci.

MaelREBOUX commented 7 years ago

Ca c'est la conséquence de #201 : cette parcelle est référencée dans la matrice foncière (majic) mais elle n'existe plus sur le plan (edigeo).

jusabatier commented 7 years ago

Ce problème ne vient pas que de #201 car en utilisant la navigation privée il se résoud. Il n'est pas lié à une parcelle en particulier, lorsqu'un utilisateur le rencontre elles sont toutes affectées et il est obligé de passer en navigation privée ou de vider completement cache, cookies, données de navigations via Ctrl+Shift+Suppr.

jusabatier commented 7 years ago

Je viens enfin de trouver d'où vient ce problème, et je pense que toutes les plateformes sont concernées :

Ce bug survient lorsqu'un utilisateur a une résolution trop faible sur son écran, ou bien que le niveau de zoom de son navigateur est trop élevé, ce qui semble faire planter le rendu de la fenêtre de fiche d'info.

MaelREBOUX commented 7 years ago

OK : je le reproduit aisément ici.

landryb commented 7 years ago

@jusabatier : peux tu préciser 'résolution trop faible' ou 'niveau de zoom trop elevé' ? Selon toi, c'est la "place disponible sur l'écran" qui n'est pas suffisante pour afficher la fenetre ? C'est un peu empirique pour rechercher ce qui peut causer ce souci....

jusabatier commented 7 years ago

Je pense en effet que c'est la place disponible sur l'écran qui pose problème.

Dans le cas d'une résolution trop faible, le zoom normal ne dispose pas d'assez de place et déclenche le bug.

Dans le cas d'un utilisateur qui met son zoom à 130%, cela a le même effet qu'une résolution trop faible.

Le bug n'est pas difficile à reproduire, il suffit de zoomer à 130~150% et d'essayer d'afficher la fiche d'info d'une parcelle.

jusabatier commented 7 years ago

Je relance cette issue car j'ai une secrétaire de mairie qui a été opérée des yeux et qui n'y voit du coup pas très bien.

Elle utilise donc des résolutions assez faible pour pouvoir lire les écritures dans ses logiciels.

Il va donc me falloir trouver assez rapidement comment solutionner ce bug :/

Personne n'a une idée de ce qui peut provoquer ce comportement avec ExtJS ?

@landryb @pierrejego @fvanderbiest

landryb commented 7 years ago

Le problème principal, c'est qu'on utilise une version d'ExtJS dépréciée depuis 5+ ans.. et que du coup, bon courage pour trouver des 'spécialistes' pour cette version.

jusabatier commented 7 years ago

J'ai trouvé comment résoudre ce problème

Cf. https://github.com/georchestra/cadastrapp/pull/304

@pierrejego pour valider