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

Favicon with number of unread articles #539

Closed
Alkarex opened this issue Jul 17, 2014 · 37 comments
Closed

Favicon with number of unread articles #539

Alkarex opened this issue Jul 17, 2014 · 37 comments
Labels
Milestone

Comments

@Alkarex
Copy link
Member

Alkarex commented Jul 17, 2014

As suggested by @Alwaysin #536 (comment)

Good reference http://stackoverflow.com/questions/6296574/dynamic-favicon-with-count-like-gmail

@Alkarex Alkarex added this to the 0.9.0 milestone Jul 17, 2014
@aledeg
Copy link
Member

aledeg commented Jul 17, 2014

That could be a nice feature :)

Alkarex added a commit that referenced this issue Aug 1, 2014
#539
Works in Firefox 32 and Opera 12. Does not work in IE 11 but without
error.
We should test if icons still work in many contexts such as placing a
shortcut on the desktop of various platforms.
@Alkarex Alkarex modified the milestones: 0.9.0, 0.8.0 Aug 1, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 1, 2014

First attempt done 8e5d98c
Testing needed, in particular possible regressions when creating shortcut icons on various platforms.

@Alkarex
Copy link
Member Author

Alkarex commented Aug 1, 2014

I forgot the screenshot
42

Alkarex added a commit that referenced this issue Aug 2, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 2, 2014

Now compatible with large numbers.
Tentatively uses SI prefixes, for instance 256 unread articles are shown as 2h+, and 4096 unread articles are shown as 4k+. Stops at 10k+.

@aledeg
Copy link
Member

aledeg commented Aug 2, 2014

👍
pas encore testé mais c'est intéressant comme fonctionnalité

Alkarex added a commit that referenced this issue Aug 2, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 2, 2014

New approach for large numbers. Examples:

  • 206 unread -> shown as 206
  • 2008 unread -> shown as 2k
  • 20100 unread -> shown as 20k
  • 200500 unread -> shown as E5

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 2, 2014

Excellent !!! 👍

Firefox - comportement OK :
2014-08-02 12_44_20- 23 vos flux rss 23 freshrss

IE - comportement NOK (même pas la favicon FreshRSS de base - je ne sais pas si c'était comme ça avant déjà ?) :
2014-08-02 12_45_44- 23 vos flux rss 23 freshrss - internet explorer

Chrome - comportment NOK (la favicon dynamique fonctionne mais l'icône FreshRSS de fond est mauvaise) :
2014-08-02 12_44_38- 23 vos flux rss 23 freshrss

Alkarex added a commit that referenced this issue Aug 2, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 2, 2014

Patch pour Chrome: 6bbf7d5

@Alwaysin Quelle version d'IE dans la capture d'écran ? (IE11 affiche le favicon non-dynamique)

Il serait intéressant de savoir s'il y a des navigateurs où le favicon dynamique devrait être en résolution supérieure à 16×16px.

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 2, 2014

Le patch fonctionne à merveille, merci.

Pour la version d'IE c'est la 11 aussi, sur Win8.1 x64.

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 2, 2014

/clapclap

Une critique : on a maintenant jusqu'à 3 fois l'information du nombre de flux non lus selon les écrans de FFRSS. Ca surcharge inutilement les onglets je trouve.

@Alkarex
Copy link
Member Author

Alkarex commented Aug 2, 2014

@Jucgshu En effet. Les avis/votes sont les bienvenus pour voir ce qu'on fait des information d'articles non-lus au niveau de l'onglet.

@Alkarex
Copy link
Member Author

Alkarex commented Aug 2, 2014

@Alwaysin Même après avoir purgé mon cache, IE11 dans Win8.1 me donne le favicon, mais qu'à partir du second clic, ceci dit... Je ne constate pas de différence avant/après le patch de favicon dynamique :

ie

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 2, 2014

J'ai remis mes paramètres IE par défaut au cas où, ainsi que vidé le cache et compagnie, et j'ai pareil : pas de favicon. Cependant cela ne me dérange pas comme je n'utilise pas ce navigateur. C'était surtout pour aider. Du coup il faudrait des tests avec d'autres personnes pour voir si ça marche chez eux ; peut-être ai-je une configuration qui fait que ça ne marche pas seulement chez moi.

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 2, 2014

@Alkarex :
Je vois 2 possibilités a priori :
1/ Le favicon donne le nb d'articles total -> ça fait une redondance simple avec le nb précédant le nom du site.
2/ Le favicon donne le nb d'articles de la catégorie -> ça fait une redondance avec le chiffre précédant la catégorie.

Dans un cas comme dans l'autre, ya de la redondance (et je vote pour supprimer les redondances :p)

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 2, 2014

@Jucgshu voir le commentaire que j'ai fais sur un autre ticket, cité dans le premier post d'Alkarex.

Alkarex added a commit that referenced this issue Aug 3, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 5, 2014

De mon côté, je suggère de garder le nombre total d'articles non-lus dans le favicon et le supprimer du titre de l'onglet. Garder par contre le nombre d'articles non-lus de la catégorie/flux en cours au niveau du titre de l'onglet.

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 5, 2014

Oui, la fonction favicon est vraiment top comme elle est. Simplement retirer la redondance de l'onglet semble une bonne solution.

@marienfressinaud
Copy link
Member

Je valide cette idée (favicon = nombre total et garder seulement le nombre de la catégorie / flux). Par contre le favicon ne prend pas en compte les flux n'apparaissant pas dans le flux principal. Comportement voulu ?

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 6, 2014

@marienfressinaud : d'un strict point de vue personnel, le use case que j'attendrais serait plutôt de ne pas les compter : si je choisis de ne pas les afficher, c'est bien pour pouvoir les consulter quand j'en ai envie, et pas qu'ils s'imposent à moi à la lecture (au comptage ici en l'occurrence).

La remarque s'applique pour le bandeau de la catégorie "Flux principal" et pour le favicon naturellement :-)

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 6, 2014

Je pense la même chose que Jucgshu :)

@marienfressinaud
Copy link
Member

Ok, je n'ai rien dit :p

@Alkarex
Copy link
Member Author

Alkarex commented Aug 6, 2014

@marienfressinaud C'était le comportement voulu que d'afficher dans le favicon le nombre d'articles non-lus du "flux principal", mais ouvert à discussion bien sûr. C'est du coup un remplacement direct du second nombre dans le titre d'onglet.

@Alkarex
Copy link
Member Author

Alkarex commented Aug 6, 2014

P.S. : un argument pour afficher dans le favicon le nombre total d'articles non-lus est que ce nombre n'est à l'heure actuelle affiché nulle part, alors que le nombre d'articles non-lus du "Flux principal" est déjà affiché dans colonne de gauche de la page.

Alkarex added a commit that referenced this issue Aug 6, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 6, 2014

Bon, voilà une version avec le nombre total d'articles non-lus dans le favicon 6972a46

@marienfressinaud
Copy link
Member

Arf, non mais c'était pas la peine du coup, c'était juste une question et @Jucgshu a très bien expliqué pourquoi ne pas les afficher (d'où mon "Ok, je n'ai rien dit :p")

@Alkarex
Copy link
Member Author

Alkarex commented Aug 7, 2014

Ok. Au moins comme ça on peut comparer. Finalement j'aime bien le favicon affichant tous les articles non-lus.

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 7, 2014

Je viens de tester la fonction mise à jour. La seule manière d'avoir un compte à zéro est de marquer comme lus des articles que je ne souhaitais pas lire sur le moment.

Sur le fond, ce n'est pas si gênant que ça.

Le risque par contre est de remettre en question l'intérêt de la fonction de non-affichage des articles dans le flux principal (en tout cas, en bon obsessionnel-compulsif du lecteur rss qui aime voir ses compteurs à zéro, je me vois obligé de me désabonner de ces flux :p)

EDIT : je me relis, et me rends compte que j'insiste (dsl). Mais j'ai beau y penser, je ne vois que des désavantages à compter les flux "inactifs".

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 7, 2014

J'approuve à nouveau ce que Jucgshu dis...

@Alkarex
Copy link
Member Author

Alkarex commented Aug 7, 2014

Voilà, j'ai annulé mon dernier patch

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 7, 2014

👍

Alkarex added a commit that referenced this issue Aug 7, 2014
A bit more transparency in the background of the text
#539
Alkarex added a commit that referenced this issue Aug 7, 2014
@Alkarex
Copy link
Member Author

Alkarex commented Aug 7, 2014

J'ai fait un léger changement de style du texte du favicon pour plus de lisibilité, avec au passage un poil de transparence dans le fond blanc du texte.

Alkarex added a commit that referenced this issue Aug 9, 2014
Now the number of unread articles in only shown in the favicon
#539
@Alkarex
Copy link
Member Author

Alkarex commented Aug 9, 2014

Comme discuté, j'ai donc supprimé le nombre d'articles non-lus "visibles" du titre, car cela faisait doublon avec le nombre déjà indiqué par le favicon. b68c5c6
Devrait être bon pour #575

@Alkarex Alkarex closed this as completed Aug 9, 2014
@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 10, 2014

Un tout petit denier détail cosmétique : peut-on inverser le nom de la catégorie et le nombre de flux de la catégorie ? Ceci afin d'éviter d'avoir 2 chiffres l'un derrière l'autre.

C'est de l'ordre du détail mais ça devrait redonner du sens à la valeur du nombre de flux de la catégorie :-)

@Alkarex
Copy link
Member Author

Alkarex commented Aug 10, 2014

Qu'en pensent les autres ?
Je trouve plus joli avec le nombre après la catégorie, mais cela a l'inconvénient qu'il est moins souvent lisible lorsque les onglets sont trop étroits.

@marienfressinaud
Copy link
Member

D'un point de vue cosmétique je suis complètement d'accord mais il me semble qu'on avait fait ainsi pour cause de manque de place dans les onglets…

@Alwaysin
Copy link
Contributor

Oui, effectivement : #536

Après, perso je suis toujours dans le flux principal quasiment et donc je ne regarde maintenant que le favicon et n'ai plus besoin du nombre de non-lus de la catégorie en cours en premier.

@Jucgshu
Copy link
Contributor

Jucgshu commented Aug 10, 2014

Dernier argument et après promis j'arrête : considérant que le nombre de flux est entre parenthèses, on peut imaginer que favicon > catégorie > nombre entre parenthèses.

Argument moisi, mais c'est tout ce qu'il me reste :-)

@Alkarex Alkarex removed the Work in progress 🚧 Wait before merging label Sep 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants