Ajout du bouton "go to top"

[firm1]

Q	R
Correction de bugs ?	non
Nouvelle Fonctionnalité ?	oui
Tickets concernés	#882

Cette PR a pour but de rajouter un bouton go to top au site.

Note pour QA

• builder les outils front : gulp build
• vérifiez sur chaque page qu'on a un bouton a bas à droite qui nous permet de revenir en haut

Screen

[Situphen]

Le bouton serait mieux avec de la couleur, en bleu ou orange par exemple ! :)

[SpaceFox]

Justement, j'aime bien le fait qu'il soit discret.

2014-12-08 10:42 GMT+01:00 Situphen notifications@github.com:

Le bouton serait mieux avec de la couleur, en bleu ou orange par exemple !
:)

—
Reply to this email directly or view it on GitHub
#1866 (comment)
.

[Eskimon]

yep mais pourquoi pas la couleur bleu principal du site ? (même si en l’état il me gène pas)

[firm1]

Le bouton serait mieux avec de la couleur, en bleu ou orange par exemple ! :)

J'ai tenté le bleu, ça se voyait trop. Du coup j'ai opté pour la couleur la plus discrète de la palette de zds.

[artragis]

Quel doit être le comportement sur mobile que je teste ça?

[firm1]

Quel doit être le comportement sur mobile que je teste ça?

Sur mobile, le bouton est censé être à la fin (pas fixé comme sur le pc), et en cliquant dessus, tu dois remonter en haut :)

[artragis]

OK.
Actuellement, la seule chose qui me gène, c'est que le bouton apparait aussi quand tu est en haut de page et que quand tu cliques dessus, ça te fait descendre (i.e ça te fait aller juste en dessous du fil d'ariane).

[firm1]

Actuellement, la seule chose qui me gène, c'est que le bouton apparait aussi quand tu est en haut de page

ça c'est le comportement uniquement sur pc

et que quand tu cliques dessus, ça te fait descendre (i.e ça te fait aller juste en dessous du fil d'ariane).

C'est bien ce que je voulais comme comportement. Faire remonter au debut du contenu.

[artragis]

Mais quand faire remonter fait baisser, c'est bizarre. En tout cas, parfait
si c'est ce que tu voulais.

Sur mobile, par contre, ça ne marche pas. Le bouton est fixe (comme sur PC)
fait fait juste chier son monde. (testé sur WindowsPhone 8.1 Lumia 920
totalement mis à jour)

Le 8 décembre 2014 11:28, firm1 notifications@github.com a écrit :

Actuellement, la seule chose qui me gène, c'est que le bouton apparait
aussi quand tu est en haut de page

ça c'est le comportement uniquement sur pc

et que quand tu cliques dessus, ça te fait descendre (i.e ça te fait aller
juste en dessous du fil d'ariane).

C'est bien ce que je voulais comme comportement. Faire remonter au debut
du contenu.

—
Reply to this email directly or view it on GitHub
#1866 (comment)
.

[firm1]

Sur mobile, par contre, ça ne marche pas. Le bouton est fixe (comme sur PC)

ça c'est pas voulu par contre. Je vais essayer de corriger ça.

[Situphen]

Je verrai bien la flèche plutôt en orange et placé entre le contenu et la sidebar, un peu comme ici :

Aussi, ce serait cool la flèche ne s'affichait pas quand il n'y a pas besoin de scroller ! Sinon, merci pour cette fonctionnalité :)

[firm1]

Perso je n'aime vraiment pas le orange.

Je l'ai mis sur la droite, car il y'a du vide là bas, et ça me semble être l'endroit ou on pose la souris (pour les droitiers) quand on veut la laisser sur le coté.

[Situphen]

Il n'y a du vide que sur les grandes résolutions, pas sur les petites !
Le 8 déc. 2014 19:03, "firm1" notifications@github.com a écrit :

Perso je n'aime vraiment pas le orange.

Je l'ai mis sur la droite, car il y'a du vide là bas, et ça me semble être
l'endroit ou on pose la souris (pour les droitiers) quand on veut la
laisser sur le coté.

—
Reply to this email directly or view it on GitHub
#1866 (comment)
.

[firm1]

Il n'y a du vide que sur les grandes résolutions, pas sur les petites !

Sur les petites resolution, le bouton reste à la fin.

PS : j'ai corrigé un peu en affichant le bouton seulement quand on a scrollé vers le bas.

[gustavi]

Je préfère le noir, c'est plus sobre. Sinon c'est bien de ne l'afficher qu'au scroll, bien vu !

[sandhose]

Au lieu d'un fadeIn/out à coup de jquery, il serait préférable d'ajouter et de virer une classe (genre .visible), et gérer ça en css... C'est bien plus performant

[sandhose]

+1 pour le noir ; y'a pas une petite erreur dans un nom de fichier ? back-top-top.js? :p

[Situphen]

Cette PR règle l'issue #882 qu'il faudra fermer quand ce sera mergé !

EDIT : merci j'ai mis le ticket à jour (gustavi)

[firm1]

Bon comme je le disais au ZestMeeting, je vais faire une longue pause sur le dev.

Du coup je ferme cette PR pour ne pas polluer la zone des PR, et je me decharge des issues associés.

La branche reste dispo sur mon depot pour ceux qui veulent s' en inspirer.

[Eskimon]

C'est con c'était presque bon ca non ?

[gustavi]

Oui ...

[artragis]

Je up tout pour que quelqu'un reprenne !