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

Titres qui dansent / en gras #534

Closed
Alwaysin opened this issue Jul 11, 2014 · 36 comments
Closed

Titres qui dansent / en gras #534

Alwaysin opened this issue Jul 11, 2014 · 36 comments
Labels
UI 🎨 User Interfaces
Milestone

Comments

@Alwaysin
Copy link
Contributor

Bonjour,

Je viens de mettre à jour mon installation FreshRSS avec la branche dev, et je vois qu'au passage sur un article, les titres "dansent", ie. se décalent vers le bas pour se re-décaler vers le haut quand on n'a plus la souris dessus. Est-ce un comportement voulu ?
Car je n'aime pas du tout ^^ En plus il n'y a ça que sur les thèmes flat design et original, pas sur le dark.

De plus, l'écriture des titres a changée. Ils sont en gras, voire de taille supérieure, et il y a peut-être même eu un changement de police ?

/Adrien.

@aledeg
Copy link
Member

aledeg commented Jul 12, 2014

Je viens d'essayer sur mon instance de dev de FreshRss et je n'arrive pas à reproduire le comportement que tu décris. Es tu sur d'avoir rafraîchi le cache de ton navigateur (ctrl + F5)?
Les thèmes ont beaucoup changés dernièrement et peut être as tu un mélange de nouvelles règles et d'anciennes règles dans ton cache.

@Alkarex Alkarex added this to the 0.8.0 milestone Jul 12, 2014
@Alwaysin
Copy link
Contributor Author

Avec ctrl + F5, j'obtiens le même comportement. En revanche, en testant sous Firefox (je n'ai jamais visité mon instance de FreshRSS avec), cela ne me le fait pas (mais les titres restent en gras avec potentiellement une police différente)
Peut-être un problème sur Chrome ? Si quelqu'un peut tester... (branche stable)

Ou peut-être faut-il vider un cache niveau serveur ?

@Jucgshu
Copy link
Contributor

Jucgshu commented Jul 12, 2014

Je ne pense pas que ce soit Chrome : j'ai le même comportement avec Firefox (FFRSS beta).

@Alkarex Alkarex added the UI label Jul 13, 2014
@Alkarex
Copy link
Member

Alkarex commented Jul 13, 2014

J'ai pu en effet reproduire le problème avec Chrome 36, thème Origine dans /dev, mais pas avec Firefox 31 ni IE 11.

Alkarex referenced this issue Jul 14, 2014
- Based on the new template
- Should remplace Origine later
- It is still not finished, there are a lot to do
@Alkarex
Copy link
Member

Alkarex commented Jul 14, 2014

La règle CSS responsable de cet effet est 1fdd76b#diff-b8f8e589bfbc82ec59bcd841ba9df583R650 dans origine.css et flat.css :

.flux:not(.current):hover .item.title {
    top: 1px;
}

Vous pouvez la supprimer manuellement en attendant.

@marienfressinaud : Est-ce que cet effet est volontaire ?

@marienfressinaud
Copy link
Member

Bonjour à tous,

les titres "dansent", ie. se décalent vers le bas pour se re-décaler vers le haut quand on n'a plus la souris dessus. Est-ce un comportement voulu ?

alors effectivement l'effet est totalement involontaire ! Mais comme ça a été noté, cela n'arrive pas sous tous les navigateurs et cette règle est justement là pour corriger le problème sous Firefox.

J'essayerai de corriger ça dans la semaine (je suis actuellement en congé prolongé donc pas dispo). Il me semblait pourtant avoir essayé sous Chromium / Opera / Epiphany, je pensais qu'il n'y avait pas de soucis :(

De plus, l'écriture des titres a changée. Ils sont en gras, voire de taille supérieure, et il y a peut-être même eu un changement de police ?

Pour le titre gras, cela n'a normalement pas changé. Le gras est appliqué pour les articles non lus ou sélectionnés.

La taille de police a effectivement était grossie car on m'a remonté que les titres étaient illisibles sinon. Cela dépend de la vue de chacun évidemment, mais j'avoue que désormais je trouve cette taille bien plus agréable.

Les thèmes ont beaucoup changés dernièrement et peut être as tu un mélange de nouvelles règles et d'anciennes règles dans ton cache.

Normalement il ne devrait pas y avoir de "mélange", j'ai fait attention à ça justement :)

@Alwaysin
Copy link
Contributor Author

Ok merci pour la réponse !

Si tu n'as pas le temps de corriger cela à court terme ce n'est pas grave, je peux vivre avec quelques temps hein :)

Sinon quel est le moyen pour moi de revenir à la taille des titres comme avant ? Probablement changer une ligne de code, j'imagine.

@Alkarex
Copy link
Member

Alkarex commented Jul 14, 2014

A priori, en supprimant cette règle, cela ne "danse" pas dans les navigateurs que j'ai sous la main : Firefox 31 (Win8), Firefox 30 (Ubuntu), IE11, Chrome 36.

Alkarex added a commit that referenced this issue Jul 14, 2014
#534
Tested on Firefox 31 (Win8), Firefox 30 (Ubuntu), IE11, Chrome 36
@Alkarex
Copy link
Member

Alkarex commented Jul 14, 2014

C'est corrigé dans /dev pour les versions actuelles des principaux navigateurs, mais pas testé pour les versions plus anciennes.

@Alwaysin
Copy link
Contributor Author

Je viens de tester la mise à jour et c'est pareil, mais quand je vais voir le fichier concerné je remarque qu'il n'y a pas les changements à l'intérieur.

Serait-ce le zip qui se met à jour par exemple une seule fois par jour ? Car j'utilise https://github.com/marienfressinaud/FreshRSS/archive/dev.zip habituellement, je ne sais pas si c'est la bonne méthode.

@Alkarex
Copy link
Member

Alkarex commented Jul 14, 2014

Je viens de regarder le zip et c'est bien bon (il est normalement mis à jour dans les secondes qui suivent un changement).
Les changements sont comme visibles sur 5d7deb6
Penser à faire un changement quelconque comme marquer un article lu/non-lu pour s'assurer de ne pas avoir une version en cache.
Si le problème persiste, avec quel thème, et quel navigateur est-ce ?

@Alwaysin
Copy link
Contributor Author

Ok, au temps pour moi : j'avais gardé un vieux dev.zip dans mon répertoire, et wget m'a créé un dev.zip.1 (ce qui est tout à fait logique, mais j'en manque parfois :)). Et donc, en décompressant le dev.zip comme à mon habitude, cela ne pouvait pas me donner les mises à jour...

Je confirme donc maintenant que les changements enlèvent le comportement en question sur Chrome 35 stable sur Win8 :D

Merci !!

@Alwaysin
Copy link
Contributor Author

Pour les titres trop gros, je suis passé de 0.9 à 0.8rem et ça me convient très bien !
Une idée éventuelle pour que je puisse conserver cette modification à travers les MAJ ? (créer un thème personnalisé j'imagine, mais je ne sais pas trop comment faire :/)

@aledeg
Copy link
Member

aledeg commented Jul 18, 2014

Si tu n'as changé que ça, je ne me fatiguerais pas à faire un nouveau thème. J'appliquerais la modification à chaque fois.
Par contre, tu peux l'automatiser en créant un script qui va ajouter ta règle à la fin du fichier CSS.

@Jucgshu
Copy link
Contributor

Jucgshu commented Jul 18, 2014

Peut-être que des options dédiées aux tailles de police sur le même modèle que la largeur : petites, normales, grosses ?

Je vais créer une issue à tout hasard.

@aledeg
Copy link
Member

aledeg commented Jul 18, 2014

en fait le mieux serait de pouvoir charger un fichier CSS customisé en plus d'un thème.
De cette manière, on pourrait modifier juste quelques éléments d'un thème en conservant ses modifications lors des mises à jours. En fait, ce fichier customisé pourrait même être utilisé sur d'autres thèmes.

@marienfressinaud
Copy link
Member

@Alwaysin > est-ce qu'en changeant seulement (c'est-à-dire en remettant le 0.9 d'origine pour la taille des titres) le font-size: 100%; du body en font-size: 90%; (dans le fichier template.css) cela te convient ?

Cette modification change la taille de l'ensemble des polices mais ça m'arrangerait pour pouvoir gérer différentes tailles de polices.

@Alwaysin
Copy link
Contributor Author

Je trouve qu'avec la font-size à 90% ça fait un peu trop petit, mais ça ne me dérange pas plus que ça. En fait c'était vraiment les titres que je trouvais trop gros et qui m'"agressaient" les yeux.

De plus, le fait de réduire la police ne donne aucun gain de place, les délimiteurs restant à leur place. Ça fait juste de l'espace utilisé en moins...

@Alwaysin
Copy link
Contributor Author

Dancing titles are back on Chrome 41 on Linux :)

@Alwaysin Alwaysin reopened this May 19, 2015
@Alkarex
Copy link
Member

Alkarex commented May 19, 2015

@Alwaysin I have not tested yet, but is it the same problem with all themes? And no problem with Firefox in your case?

@Alwaysin
Copy link
Contributor Author

Yes, the same with all 6 themes and not happening in Firefox.

@Alkarex
Copy link
Member

Alkarex commented May 19, 2015

@Alwaysin I cannot reproduce the problem... So please double-check that you are using the /dev branch, and with the current version of Chrome (43).

I have just tested on Ubuntu 15.04 (virtual machine):

  • Firefox 38 (default browser): all good
  • Chromium 41: no mouse-hover effect at all (even on other Web sites), so no dancing
  • Google Chrome 43 (manual install, because it is not in the repositories): all good
  • Web Epiphany 3.14.2: all good
  • Midori 0.4.3: all good

And on Windows 8:

  • Internet Explorer 11 (default browser): several problems, but fine regarding mouse-hover
  • Firefox 38: all good
  • Chrome 43: all good
  • Opera 29: all good

@Alkarex
Copy link
Member

Alkarex commented May 19, 2015

#845

@Alwaysin
Copy link
Contributor Author

I've just updated using the automatic system :

Your current version of FreshRSS is the 1.2-dev.
Last verification: May 19th 2015 at 21:40

I've also tested on Win8.1 with Chrome 43 and again I got the dancing titles.

I have no extensions.

I made a diff on p/themes/base/ folder and a css of a theme (local files with GitHub files) and there is no change, so it shouldn't be from something I could have changed here (if there are other files I should check please tell me).

Next step is to try with a new FreshRSS installation I guess. I had planned to do it for testing beta 1.1.1 anyway, so let's give it a try :)

@Alkarex
Copy link
Member

Alkarex commented May 19, 2015

I have to admit that I have not looked at how the automatic update system is working. Do you have the same files as https://github.com/FreshRSS/FreshRSS/tree/dev/p/themes ?

@Alwaysin
Copy link
Contributor Author

Positive for the subfolders icons/ fonts/ base-theme/ and Origine/, the theme I'm using right now (others shouldn't be able to screw things when not using their theme, right?). All files last modified May 19th at 21:40 (tending towards the fact they haven't been modified).

I've just tried with IE and it does it also.

But contrary to the initial post, it does not go downwards when the mouse is over and then goes back to the initial position when the mouse goes out, but it is reversed : it goes upwards then downwards when moving the mouse out of the title.

@Alwaysin
Copy link
Contributor Author

I couldn't reproduce this behaviour on a fresh installation. Problem "solved" I guess :)
I've probably tampered with something on my old installation...

@Alwaysin
Copy link
Contributor Author

What is strange is that this beahviour is always coming back. When I do a fresh installation it is not there, but always comes back with time even though I do not touch anything server-side...

@Alkarex
Copy link
Member

Alkarex commented Oct 22, 2016

@Alwaysin Quel navigateur / plateforme / thème / version de FreshRSS ?

@Alwaysin
Copy link
Contributor Author

Alwaysin commented Oct 22, 2016

Sur Chrome, IE 11 et Edge sous Windows 10. Ne se produit pas sur Firefox. J'ai essayé tous les thèmes et le problème était toujours là. Je suis sur /dev à jour des derniers commits.

@Alkarex
Copy link
Member

Alkarex commented Oct 22, 2016

@Alwaysin Je n'arrive pas à reproduire le problème (testé sous Chrome, IE11, Edge sous Windows 10). Pourriez-vous faire une petite vidéo / gif, et/ou m'envoyer par courriel une adresse où je pourrai voir le problème ?

@Alwaysin
Copy link
Contributor Author

Chrome :

freshrss_dancing_titles_chrome

Firefox :

freshrss_dancing_titles_firefox

Email envoyé :)

@Alkarex
Copy link
Member

Alkarex commented Oct 22, 2016

J'ai réussi à reproduire le problème : c'est quand les dates sont en anglais, à cause des exposants

@Alkarex
Copy link
Member

Alkarex commented Oct 22, 2016

Je pense que j'ai trouvé une solution #1340
Pourriez-vous tester ?

Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Oct 22, 2016
@Alwaysin
Copy link
Contributor Author

Je confirme que ça fonctionne bien, merci beaucoup 👍

@Alwaysin
Copy link
Contributor Author

Je disais que c'était pas si important que ça mais ça repose beaucoup les yeux finalement :D

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