zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 160 forks source link

Nouvellel favicon #5032

Open A-312 opened 6 years ago

A-312 commented 6 years ago

Ouais, je trouve aussi que ça ne rend pas très bien, en plus ça change tout le temps l’icône associée au site dans ma barre de favoris.

Ne pas changer le favicon de la page dans le code HTML. Il faut faire le changement après le chargement de la page (ready ou load, au choix).

@rezemika une raison pour avoir préféré le code HTML plutôt qu'une solution en js ?

Il suffirait donc d'ajouter un data-attr ou de bêtement compter le nombre d’élément dans le dropdown des notifications au menu notification pour les détecter.


Pour améliorer la chose : Avec le localstorage, il faudrait mettre à jour les notifications dans les autres onglets. Par exemple : En tenant à jour un objet avec les notifications non lus et avec un setInterval(... , 1000) --> 1000 ou plus

rezemika commented 6 years ago

@rezemika une raison pour avoir préféré le code HTML plutôt qu'une solution en js ?

En fait c'est très simple : je n'avais pas pensé à cet "effet de bord" sur les icones des favoris.

Cela dit, je pense que le débat qui a lieu sur le forum est symptomatique d'un manque de communication entre les développeurs (et moi le premier) et la communauté en général. J'ai tendance à me dire (et je ne suis peut-être pas le seul) qu'un ticket posté sur Gituhb est quelque chose qui fait à peu près consensus auprès de la communauté et qu'il ne reste plus qu'à intégrer au site. AMHA, il serait pertinent d'essayer de trouver une solution pour éviter que de tels problèmes se reproduisent (ce qui, en plus, pourrait peut-être ramener de nouveaux contributeurs).

A-312 commented 5 years ago

Ca fait 1 an qu'on le garde... Faut vraiment qu'on le fasse !

artragis commented 5 years ago

Merci de ne pas toucher au projet v29 comme ça.