search

JForsaken / protosketcher-frontend

Protosketcher's frontend application.
0 stars 0 forks source link

Fix usage of ducument.getElementById #122

Closed jrainville closed 7 years ago

jrainville commented 7 years ago

Should be the end of this: https://github.com/JForsaken/protosketcher-frontend/issues/83 Removed usage of document.getElementById by use of ref Had to enable a flag in Shape and Text called withRef, because since I put refs on the component and not a Dom element, without that flag, we could not get the Component but only a weird Component.

jrainville commented 7 years ago

C'est réglé @JForsaken . C'est un peu stupide. Ça fonctionne de mettre dans un tableau/object, c'est juste que selection.jsx s'update pas directement (pas de hot reload), donc quand je changeait la façon d'accéder au Component, il me disait, au non il existe pas, mais juste reloader faisait en sorte que tout fonctionne.

JForsaken commented 7 years ago

@jrainville t'es tu sur t'as les bons fichiers? Jai rename selection et un autre, ils etaient des jsx au lieu de des js justement... peut etre que t'as 2 fois le meme fichier ti checkeras, ca ressemble a ca maintenant img_0750

jrainville commented 7 years ago

Oui j'ai les bons fichiers, c'est mon erreur, j'ai marqué jsx, qaund en fait je voulais dire js. N'empêche, ça fonctionne live, mais le reload le prenais pas anyway. Je crois que c'est parce que c'est juste un helper et non un component qui doit être re-render. J'ai eu le même problème avec event.js

JForsaken commented 7 years ago

@jrainville ouais t'as raison ca m'a deja fait aussi quand c'est pas un component.. perfect then merge moé soh