francejs / francejs.github.com

Site web
http://francejs.org
13 stars 26 forks source link

Nouvel emplacement pour la carte #74

Closed victordarras closed 10 years ago

victordarras commented 10 years ago

Vraiment bien l'idée de la map , je me suis dit que ça serait plus sympa de la voir un peu plus grand et qu'elle soit plus intégrée dans le corp du site , c'est plus immersif. Du coup je l'ai mise en bas , là ou ça ne dérange pas. Voilà à quoi ça ressemble : francejs

MoOx commented 10 years ago

My 2cts: l'idée de base est bien, mais perso je trouve ça too much sur le coup, surtout que on a pas vraiment besoin d'autant de largeur. Après c'est vrai que ça peut être pas mal de la mettre en plus grand à un endroit c'est sur.

Also @victordarras ici ca change de dézoomé sur la carte déjà https://github.com/francejs/francejs.github.com/pull/73

victordarras commented 10 years ago

Ca peut faire un peu too much yep. C'est déjà bien mieux dézoomé. J'attends de voir un peu ce que les évolutions donnent,

kud commented 10 years ago

On en est où avec cette PR ? A noter qu'elle n'est pas mergeable en l'état.

MoOx commented 10 years ago

La question est: est ce que ça intéresse cette vue de la carte où pas ? :) Perso pourquoi pas, même si je trouve la vue actuelle suffisante en l'état. Note: Le dernier commit au titre un peu étrange inclus un conflit (cf dernier fichier modifié)

kud commented 10 years ago

Y'a trop de changement qui ont été fait là. Faut rebaser. Et à mon sens, la map est très bien comme ça.

kud commented 10 years ago

J'ai aggrandi la carte et je l'ai mise en bas. Ca me parait pas mal comme ça. A réouvrir si nécessaire.