spip#5343 suppression du filtre |lignes_longues maintenant géré en css

@Jack31 a ajouté 1 révision : 90e4942e

@b_b a évalué le commit 90e4942e :

diff --git a/CHANGELOG.md b/CHANGELOG.md
--- /dev/null
+++ b/CHANGELOG.md
@@ -0,0 +1,7 @@
+# Changelog
+
+

Il manque un ## [Unreleased] ici cf https://git.spip.net/spip/spip/src/branch/master/CHANGELOG.md

@b_b a rejeté cette PR (90e4942e)

@b_b a évalué le commit 90e4942e :

diff --git a/css/theme.css b/css/theme.css
--- a/css/theme.css
+++ b/css/theme.css
@@ -207,6 +207,8 @@ ul.choix_mots li label { font-weight: normal; font-size: 1em; line-height: 1.25;
 .comment-meta abbr { border: 0; }
 .comment-meta a { text-decoration:none;}
 .comment-content .hyperlien { padding: 0; background: transparent; font-weight: normal; }
+.comment-texte p {word-wrap: anywhere;}

ça ne mange pas de pain, mais des espaces après et avant les accolades ne feraient pas de mal :)

@b_b a évalué le commit 90e4942e :

diff --git a/css/theme.css b/css/theme.css
--- a/css/theme.css
+++ b/css/theme.css
@@ -208,2 +208,4 @@
 .comment-meta a { text-decoration:none;}
 .comment-content .hyperlien { padding: 0; background: transparent; font-weight: normal; }
+.comment-texte p {word-wrap: anywhere;}
+.description {word-wrap: break-word;}

ce sélecteur me semble trop "évasif" ça risque d'impacter d'autres blocs

@Jack31 a évalué le commit 90e4942e :

diff --git a/css/theme.css b/css/theme.css
--- a/css/theme.css
+++ b/css/theme.css
@@ -208,2 +208,4 @@
 .comment-meta a { text-decoration:none;}
 .comment-content .hyperlien { padding: 0; background: transparent; font-weight: normal; }
+.comment-texte p {word-wrap: anywhere;}
+.description {word-wrap: break-word;}

Je propose de remplacer par comment-description ce qui donnerait ligne 21 et 46 de dist/inclure/forum <div class="#EDIT{texte} comment-content comment-description">

(et je corrige le changelog et les espaces)

@erational a évalué le commit 90e4942e :

diff --git a/css/theme.css b/css/theme.css
--- a/css/theme.css
+++ b/css/theme.css
@@ -208,2 +208,4 @@
 .comment-meta a { text-decoration:none;}
 .comment-content .hyperlien { padding: 0; background: transparent; font-weight: normal; }
+.comment-texte p {word-wrap: anywhere;}
+.description {word-wrap: break-word;}

bof ... pourquoi avoir 2 classes ?
Ce n'est pas très signifiant, autant réduire à <div class="#EDIT{texte} comment-content">

@Jack31 a ajouté 1 révision : 56d6a8e5

@Jack31 a évalué le commit 90e4942e :

diff --git a/css/theme.css b/css/theme.css
--- a/css/theme.css
+++ b/css/theme.css
@@ -208,2 +208,4 @@
 .comment-meta a { text-decoration:none;}
 .comment-content .hyperlien { padding: 0; background: transparent; font-weight: normal; }
+.comment-texte p {word-wrap: anywhere;}
+.description {word-wrap: break-word;}

J'ai mis à jour la PR

@Jack31 a fait référence à cette PR depuis un commentaire de spip/spip!5353 spip/#5343 Suppression du filtre lignes_longues

Presque parfait @Jack31 par contre, pourquoi avoir retiré la classe description dans le source ? Ça risque de péter des trucs chez les gens qui se basait sur la présence de celle-ci. Amha, il faut la laisser.

@Jack31 a ajouté 1 révision : 2962e6bc

Tiens, a priori, overflow-wrap remplace le terme world-wrap (un alias) https://caniuse.com/?search=overflow-wrap indique qu’on peut l’utiliser sans soucis

@marcimat a fait référence à cette PR depuis !4864 spip/spip#5343 suppression du filtre |lignes_longues maintenant géré en css #4853

Voir #4864

@marcimat a fermé cette PR