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 balise title pour afficher les titres #322

Closed
wants to merge 1 commit into from

Conversation

mathieumd
Copy link

Pour éviter que le text-overflow:ellipsis ne rende les longs titres illisible dans les fenêtres de faible largeur.

Pour éviter que le text-overflow:ellipsis ne rende les longs titres illisible dans les fenêtres de faible largeur.
@Alkarex
Copy link
Member

Alkarex commented Dec 18, 2013

Merci.
Je trouve qu'il serait bien de pouvoir éviter d'afficher la bulle de titre lorsque le titre n'est pas tronqué.
C'est bien sûr faisable en JavaScript, mais il serait bien d'avoir une solution en CSS.
Il faut que j'y réfléchisse.

@mathieumd
Copy link
Author

Complètement d'accord !

Alkarex added a commit that referenced this pull request Dec 18, 2013
Alkarex added a commit that referenced this pull request Dec 18, 2013
Alkarex added a commit that referenced this pull request Dec 18, 2013
@Alkarex
Copy link
Member

Alkarex commented Dec 18, 2013

Je viens de faire un essai rapide 8f002eb et corrections dans 5c779a0 et 3280b14
Il fallait finalement quelques changements de plus...
Je testerai mieux demain.
Qu'en pensez-vous ?

hover_

@mathieumd
Copy link
Author

Bonne idée ! Ça a l'air pas mal.

Mais quid des très longs titres ? Si ils s'affichent sur 2+ lignes, alors c'est parfait.

Je n'ai pas pu le tester à cause de l'import impossible de la branche dev (cf. #228 (comment))

@Alkarex
Copy link
Member

Alkarex commented Dec 19, 2013

Pour l'instant, c'est sur une seule ligne, mais on gagne déjà l'espace
occupé par la date, lorsque nécessaire

@Alkarex
Copy link
Member

Alkarex commented Dec 19, 2013

@mathieumd l'installation de /dev devrait maintenant fonctionner à nouveau

@ghost ghost assigned Alkarex Dec 19, 2013
@mathieumd
Copy link
Author

J'ai pu tester en grandeur réelle, et c'est bien mieux qu'avant.
À mon avis il resterait à fignoler un peu plus l'apparence :
Par exemple, je pense au fait que quand la ligne est plus longue que la largeur de l'écran, la barre de défilement horizontale apparaît.
menu_2013-12-20-001
Ainsi qu'au long espace blanc à droite du titre et au manque d'un séparateur explicite (dégradé transparent ou autre) quand la date est partiellement recouverte par le titre.
menu_2013-12-20-002

Mais je chipote : le besoin de base est largement satisfait. Merci ! :)

@mathieumd mathieumd closed this Dec 20, 2013
@Alkarex
Copy link
Member

Alkarex commented Dec 21, 2013

Ok :-)
Au passage, FreshRSS permet de choisir quelles icônes afficher, ce qui est en particulier utile pour les petites largeurs d'écran. Dans mon cas, sur la ligne du haut, je n'affiche pas l'icône lu/pas-lu ni l'icône pour ouvrir le lien à droite.

Alkarex added a commit that referenced this pull request Dec 21, 2013
Changements bordure droite
#322
@Alkarex
Copy link
Member

Alkarex commented Dec 21, 2013

Je viens de faire une petite modification pour la bordure de droite

@Alkarex
Copy link
Member

Alkarex commented Jan 23, 2014

Suite dans #385

Alkarex added a commit that referenced this pull request Jan 23, 2014
+ correction bug "..."
#385
#322
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants