JForsaken / protosketcher-frontend

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

Side panel improvements #112

Closed jrainville closed 7 years ago

jrainville commented 7 years ago

J'ai changé le side panel en enlevant le Drawer, car il ne permettait pas de bouger le reste du contenu. Ce que je veux dire, c'est que Drawer doit être au dessus du contenu. Bref, maintenant, la zone de draw est toujours centrée et le menu pousse la zone. C'est ben du CSS bizarre alors bonne chance pour review, mais au final c'est vraiment plus clean comme user experience.

franck36 commented 7 years ago

Dude...j'ai déja fait sa dans ma branche

jrainville commented 7 years ago

Nooooon. Fuck. On aurait du se parler, mais c'est ma faute, j'ai pas fait ça avant pour t'accelerer et je m'attendais pas à ce que tu t'attardes à ça... On fait quoi?

franck36 commented 7 years ago

On compare les deux. On peux p-e merger ta branche dans la mienne.

jrainville commented 7 years ago

J'ai testé ton code et je crois qu'on devrait prendre le mien, car avec display: table-cell (ce que j'ai utilisé), le height du menu est toujours à 100%. Aussi, j'ai enlevé la dépendance à Drawer wui ne sert plus à rien, car je gère l'animation moi-même. Qu'est-ce que tu en penses?

franck36 commented 7 years ago

J'ai pas checké le code, alors si ton code est mieux, sa me déranges pas mais : 1- Quand sa load, on voit que le menu est plus petit est apres il prends 100%. 2- L'animation pour le expand marche pas bien : sa expande au max sans animation et apres sa flicker. 3- Le scrollbar est rendu en bas du menu au lieu de seulement dans la zone de drawing.

Faudrais fixer sa avant, car ma version ne fait pas sa. Aussi, vu que j'ai fait plusieurs mod, il faudrait merger dans ma branche pour pas que sa fuck up le reste au moment de merger ma branche.

jrainville commented 7 years ago

Ah ouais c'est vrai j'avais oublié. Je viens de voir un autre gros problème du tiens, Si l'écran est petit en largeur, on peut pas scroller pour voir le reste du workspace. Je vais fixer ce que tu as spécifier par contre.

franck36 commented 7 years ago

OK. Si tu veux merge, je ne modifie plus le side-panel, je travaille sur le contenu alors on devrait pas avoir de conflits.

jrainville commented 7 years ago

J'ai fix la plupart des patentes. La seule affaire c'est que pour la scroll qui est par dessus, il n'y a rien à faire. Même dans le tiens je crois que c'était comme ça. C'est pas trop grave je crois.

franck36 commented 7 years ago

C'est tu moi ou les animations sont weird. C'est full rapide. J'ai essayé de mettre les animations normales, soit : 450ms cubic-bezier(0.4, 0.0, 0.2, 1), mais c'est le contenu qui disparait et ensuite le paneau one shot. Aussi, la grandeur du side panel change en fonction de la grandeur de l'écran.

jrainville commented 7 years ago

Ah oui j'ai mis le width en pourcentage. Je vais essayer avec un width fixed

JForsaken commented 7 years ago

c'est tu moi ou la simulation est petée sur ta branche?

Edit: le commit 'lint ma vie' made me chuckle

Edit mode

image

Simulation

image

jrainville commented 7 years ago

My bad. J'avais refactor le CSS pour mettre le style de .workspace dans les classes parentes, mais j'ai oublié que .workspace est aussi utilisé par Simulation mais sans les classes parentes. C'est de-refactored eksdee

jrainville commented 7 years ago

Je crois que tout devrait être beau là :tired_face:

franck36 commented 7 years ago

On checkera sa mardi voir si on est capable de se merger sans tout casser.

jrainville commented 7 years ago

:violin: