Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout de neige dans le header pour Noël (PoC, à discuter) #1911

Closed
wants to merge 1 commit into from

Conversation

sandhose
Copy link
Contributor

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? oui
Tickets concernés Forum, Zest'Meeting

Une proposition pour l'habillage de noël du site

Let it snow

C'est juste un PoC, à discuter, si ca plaît (ou non) ; c'est encore un peu buggé (genre le menu des notifs qui est décallé...), mais ça sera réglé si tout le monde est d'accord pour qu'on affiche ça le soir de noël

@sandhose sandhose added C-Front Concerne l'interface du site Question labels Dec 14, 2014
@poulp
Copy link
Contributor

poulp commented Dec 14, 2014

J'aime bien, avec un habit rouge ça serait top ;) (http://zestedesavoir.com/forums/sujet/1928/changement-de-couleur-pour-zds)

@gustavi
Copy link
Contributor

gustavi commented Dec 15, 2014

Est-ce que tu peux faire en sorte que l'option soit activable dans le settings.py ? Ça serait assez cool de la pas avoir à faire une MEP pour changer l'option.

@Eskimon
Copy link
Contributor

Eskimon commented Dec 18, 2014

On en est ou ?

@Alex-D
Copy link
Contributor

Alex-D commented Dec 19, 2014

Info : canvas ça fait rammer/planter certaines machines. Faut faire des tests sur des petites machines.

Il faut tester que le canvas ne passe pas au dessus des différents éléments, qu'il ne change rien à l'ordre des éléments focussables, etc. J'imagine que @sandhose a déjà testé ça, mais la QA doit recheck.

EDIT : Peut-être qu'il faudrait baisser l'opacité des flocons ?

Il faut aussi donner la possibilité aux utilisateurs de la mettre en pause (virer le canvas et mettre une image en background ?). Règle élémentaire d'accessibilité : tout animation doit proposer un contrôle de lecture/pause.

@sandhose
Copy link
Contributor Author

Testé sur un iPhone 3GS (2009), et tourne très bien (même mieux que la transition du menu latéral), mis à part la compatibilité du requestAnimationFrame qui est préfixé (webkit...) ; faut juste que je rajoute un polyfill ; est-ce qu'on peut considérer que c'est suffisant comme "compatibilité" ? (il a quand même 5 ans, ce téléphone :-° )

En parlant de mobile, ça pourrait être marrant d'avoir la neige qui tombe en fonction de l'inclinaison du device. Genre que la gravité soit celle du téléphone

Pour l'opacité, suffit de tweaker un peu la variable PARTICLES_COLOR... Je vais essayer ça

Pour la pause, je vois pas trop trop où placer le bouton en question...

@SpaceFox
Copy link
Contributor

Je vais violer toutes les procédures, mais bon...

Normalement la release 1.4 sort demain soir.
Est-ce qu'on ne pourrait pas y inclure ça ? ce qui nous éviterait de faire une 2ème MEP je ne sais pas quand juste pour cette histoire de neige ?

D'ailleurs, comment va-t-on le désactiver ?

@gustavi
Copy link
Contributor

gustavi commented Dec 21, 2014

Est-ce que tu peux faire en sorte que l'option soit activable dans le settings.py ? Ça serait assez cool de la pas avoir à faire une MEP pour changer l'option.
-- Moi

@Situphen
Copy link
Member

@sandhose : Il reste des trucs à faire ou pas ?

@artragis
Copy link
Member

artragis commented Jan 6, 2015

on a raté le coche, à fermer @SpaceFox .

@SpaceFox SpaceFox closed this Jan 6, 2015
@SpaceFox SpaceFox removed the En cours label Jan 6, 2015
@pierre-24
Copy link
Member

Y'aurais moyen de ressortir ça pour cet année, @sandhose ?

@sandhose sandhose deleted the let-it-snow branch January 6, 2016 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants