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

bigMarkAsRead — Texte plus gros #480

Closed
Cypouz opened this issue Apr 10, 2014 · 28 comments
Closed

bigMarkAsRead — Texte plus gros #480

Cypouz opened this issue Apr 10, 2014 · 28 comments
Assignees
Labels
UI 🎨 User Interfaces
Milestone

Comments

@Cypouz
Copy link

Cypouz commented Apr 10, 2014

Yop !

OK, le bouton « Fin des articles, tout marquer comme lu » est gros. Je n'ai rien contre, mais à ce moment-là faut assumer :) J'y pense à chaque fois que je le vois : le texte est bien trop petit, perdu dans cette immensité. Presque ridicule.

#bigMarkAsRead {
    font-size: 4em;
    font-weight: bold;
    color: #ccc;
}

#bigMarkAsRead:hover {
    color: #aaa;
}

C'est bien mieux comme ça ;)

@Alkarex Alkarex added the UI label Apr 10, 2014
@Alkarex
Copy link
Member

Alkarex commented Apr 10, 2014

Je laisse @marienfressinaud regarder pour le style :-)

@aledeg
Copy link
Member

aledeg commented Apr 10, 2014

J'ai fait un test rapide en changeant la taille avec celle proposée.
C'est agressif. Je pense que c'est mieux de le laisser tel quel.

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 8, 2014

Je suis d'accord sur le principe. Le texte est trop petit. Mais je n'ai pas testé la modification proposée pour voir si elle rend la chose vraiment trop grosse ou pas.

La bande est trop grande aussi (en hauteur), il faudrait enlever au moins un saut de ligne avant le "fin des articles" et après le "tout marquer comme lu".

@marienfressinaud marienfressinaud added this to the 0.8.0 milestone Sep 11, 2014
@marienfressinaud
Copy link
Member

Je verrai pour l'agrandir (je n'aime pas cette partie de toute façon, surtout le nouveau :hover) mais je trouve aussi que 4em c'est beaucoup trop gros.

marienfressinaud added a commit that referenced this issue Sep 16, 2014
Increase font size
Change colors in Origine

See #480
@marienfressinaud
Copy link
Member

J'ai fait une tentative de modification (police un peu plus grosse + changement des couleurs dans Origine). À vous de me dire si vous trouvez ça mieux ;)
bigmark

@marienfressinaud
Copy link
Member

(là c'est la vue au survol de la souris)

@Alwaysin
Copy link
Contributor

Niveau design moi j'aime bien mais je comprends pas ce qui justifie le fait qu'en place ça prenne l'équivalent de plus de 10 articles.

PS : quand je fais "vérifier les mises à jour", je n'ai rien qui se passe (l'URL change avec le check, mais je n'ai pas l'indication de la mise à jour à faire). Pourtant je l'ai mis à jour y'a pas longtemps avec tes derniers commits et c'est passé sans problème.

@aledeg
Copy link
Member

aledeg commented Sep 16, 2014

Personnellement, je ne l'utilise jamais, lui préférant le raccourci.
Je trouve aussi que c'est gros et surtout redondant avec le bouton du haut. J'avoue que j'aimerais avoir les barres fixes (catégories et entête) et les articles mobiles pour toujours avoir à disposition les outils.

@Alwaysin
Copy link
Contributor

+1 pour les barres fixes, ça peut être sympa, au moins pour les catégories !

@marienfressinaud
Copy link
Member

@Alwaysin > perso ça ne gène pas que ça soit gros ou non, je tatonne un peu pour trouver ce qui sera définitif à ce niveau-là. On peut faire des tests avec des tailles plus petites du coup :)

Pour les mises à jour, si la dernière date d'il y a moins d'une heure, c'est normal. Faut encore que j'améliore ça pour que ça soit explicite ;)

@aledeg > je comprends, mais je ne suis pas fan des tests que j'ai fait jusque-là. Ce qui m'irait le mieux, c'est le position: sticky qui n'est pas répandu du tout (apparu dans Firefox 32). Démo

@marienfressinaud
Copy link
Member

J'ai réduit la place prise par le bouton :

bigmark

Sinon @aledeg j'ai oublié de te répondre sur l'utilité de ce bouton : perso je l'utilise beaucoup sur mobile une fois que j'ai fini de lire les articles, j'ai juste à appuyer dessus pour passer à la suite. D'ailleurs je suis contre le menu flottant sur mobile car il prend beaucoup de place et il y a déjà le menu de navigation en bas de l'écran.

Sur PC comme toi, je préfère le raccourci, mais je suis d'avis de laisser ce bouton quand même qui peut être pratique.

marienfressinaud added a commit that referenced this issue Sep 16, 2014
@Alwaysin
Copy link
Contributor

J'aime mieux la taille comme ça :)

Après je reste un peu choqué par la taille du texte, on dirait que c'est plus gros que le titre des articles (je ne sais pas si c'est un effet d'optique ou pas). Mais je vais pas chipoter dessus ;)

Ok pour les maj ! Du coup faudrait juste au moins ajouter un "pas de mise à jour à faire" non ? Parce que là on croit que la fonctionnalité n'est pas implémentée :/

@marienfressinaud
Copy link
Member

Ce n'est pas un effet d'optique mais j'avais cru que c'était ce que vous demandiez @Cypouz et toi. Tu disais même en août :

Je suis d'accord sur le principe. Le texte est trop petit.

:p

marienfressinaud added a commit that referenced this issue Sep 16, 2014
last_update.txt is updated even if there is no update
If last_update.txt has been modified during last hour, show a message

See #480 (comment)
@Alwaysin
Copy link
Contributor

Yep sauf que y'a un juste milieu entre pattes de mouche et pattes d'éléphant :p

Après c'est sur que tu peux pas deviner, j'aurais du tester et faire une proposition de la taille que je souhaite (comme cypouz ^^ et d'ailleurs c'est sa proposition que tu as mis ?)

@marienfressinaud
Copy link
Member

Ah non non, j'ai mis beaucoup plus petit :p Je vais réduire un peu, là je bosse sur autre chose. Au passage j'ai ajouté le message pour les mises à jour ;)

marienfressinaud added a commit that referenced this issue Sep 16, 2014
@marienfressinaud
Copy link
Member

J'ai pushé le tout, ça devrait être mieux.

@marienfressinaud marienfressinaud self-assigned this Sep 16, 2014
@Alwaysin
Copy link
Contributor

Ok super, pour moi c'est OK j'aime comme ça :)

Pour le message de mise à jour, ça se voit où ? Je crois pas l'avoir :/

@marienfressinaud
Copy link
Member

@Alwaysin peux-tu ouvrir un nouveau ticket avec un screenshot de ce que tu obtiens + éventuellement les logs FreshRSS ?

@marienfressinaud
Copy link
Member

J'ai fait une petite modif ce matin au survol :
bigmark

Je pense qu'on va pouvoir dire que c'est fini, j'aimerais juste avoir confirmation de @Cypouz que l'on peut fermer son ticket ;)

@Cypouz
Copy link
Author

Cypouz commented Sep 17, 2014

Salut !

Tout d'abord, je voudrais vous remercier pour vos avis ainsi que pour l'attention portée par Marien pour satisfaire nos demandes.

Déjà, tu réduis la problématique en réduisant la taille du bouton, ce qui n'est pas un mal. La grosse taille assumée de la police que je proposais ne venait qu'accompagnée des couleurs également proposées qui étaient justement là pour adoucir l'agressivité d'une telle taille. Ainsi, le texte sur le bouton faisait écho au span.name visible dans les bandeaux du flux.

Je trouve ta tentative d'améliorer le texte de ce bouton louable. Néanmoins, et c'est bien visible sur ta dernière capture d'écran, on sent que quelque-chose cloche avec ce bouton. Il paraît bien étrange ce bloc de texte au milieu de tout ce vide. Je pense que des demi-mesures ne pourront satisfaire personne. À un moment donné, il faut assumer un choix graphique fort et cohérent. J'en proposais un qui ne pourra pas, comme le tient, satisfaire tout le monde.

Bref, si on a fait le tour du sujet et que globalement ce nouveau look plaît à toi ainsi qu'aux autres, je trouve que c'est une bonne chose. Tu peux donc fermer le ticket ;)

Vous trouverez néanmoins ci-dessous deux propositions basées sur ce que j'écrivais plus haut, avec quelques modifications supplémentaires, pour montrer à quoi pourrait ressembler du texte gros sans forcément rendre l'ensemble plus moche. Sur la première, le bouton n'est pas couvert par la souris, ce qui est le cas sur la seconde (et oui, je sais, j'utilise la dernière version stable :).

capture du 2014-09-17 12 15 33

capture du 2014-09-17 12 16 31

@marienfressinaud
Copy link
Member

Je comprends ton problème mais il n'y a pas de solution idéale pour le moment :( Je trouve qu'augmenter la police comme sur tes screenshots juste pour combler le vide, ça ne rend pas bien.

Le mieux serait de voir comment ça a été fait ailleurs. Il me semble qu'@Alkarex avait repris l'idée de Feedly ou quelque chose comme ça ?

@Alkarex
Copy link
Member

Alkarex commented Sep 17, 2014

@marienfressinaud Je trouve la nouvelle version assez bien, sauf la couleur verte, trop criarde à mon goût et pas trop en accort avec le reste du thème, mais c'est un détail

@marienfressinaud
Copy link
Member

Ok, la couleur était un essai de ma part, je peux essayer de trouver autre chose ;)

Edit : j'ai utilisé le bleu des liens.

marienfressinaud added a commit that referenced this issue Sep 18, 2014
@marienfressinaud
Copy link
Member

Comme on n'arrivera pas à satisfaire tout le monde et que je commence tout de même à être satisfait personnellement de ce bouton, je ferme le ticket.

@Cypouz désolé mais sinon ce sera sans fin entre toi qui veut plus gros et ceux qui veulent plus petit :p

@Cypouz
Copy link
Author

Cypouz commented Sep 18, 2014

Pas de problème. C'est un point de détail. Ça pourrait me motiver pour mettre le nez dans les thèmes personnalisés.

@aledeg
Copy link
Member

aledeg commented Sep 18, 2014

Ce qui serait bien, ce serait d'avoir une option qui permette de charger un fichier css qui permettrait de personnaliser un thème sans avoir à en créer un.

@marienfressinaud
Copy link
Member

@aledeg > j'y pense depuis un moment déjà ;)

@Alkarex
Copy link
Member

Alkarex commented Sep 18, 2014

J'ai un début de code pour ça, dans le cadre des extensions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 🎨 User Interfaces
Projects
None yet
Development

No branches or pull requests

5 participants