#7 la largeur max-width par defaut devrait être supprimée

Open
opened 7 months ago by touti · 9 comments
touti commented 7 months ago
Collaborator

la largeur max-width par defaut devrait être supprimée

crayon.css

ligne 51

.crayon-html .formulaire_spip { max-width: 44rem;

la largeur max-width par defaut devrait être supprimée crayon.css ligne 51 .crayon-html .formulaire_spip { max-width: 44rem;

mille fois OUI !
...Touti tu ne ferais pas carrément une PR que ça puisse être intégré rapidement?

mille fois OUI ! ...Touti tu ne ferais pas carrément une PR que ça puisse être intégré rapidement?
touti commented 6 months ago
Poster
Collaborator

je l'ai fait à l'instant depuis l'interface en ligne mais la PR ne semble pas apparaitre

je l'ai fait à l'instant depuis l'interface en ligne mais la PR ne semble pas apparaitre
tofulm commented 6 months ago
Collaborator

je ne le vois pas également

je ne le vois pas également
touti commented 6 months ago
Poster
Collaborator

la PR est là
ed4c2620cf

et voici ce que répond cerdic

Salut Touti,

ce max-width est là depuis toujours ou presque, pour des raisons principales de lisibilité : si on édite sur un div qui est très large on se retrouve avec des lignes de texte beaucoup trop longues.

Dans la dernière modif le concernant, j'avais modifié le max-width fixe en px en le passant en rem, ce qui est plus générique
b61034d6c8

Je ne pense pas que le supprimer soit une bonne idée, c’est généralement mieux de l’avoir.
Si il y a des cas précis qui ne le justifient pas je pense qu’il faut simplement personaliser dans la css de ton site, non ?

--
Cédric


Salut Cédric,

Ehhh mais non, c'est pas cool de répondre que j'ai qu'à modifier sur "mon" site, parce qu'évidemment je ne serai pas ici si l'intérêt n'était pas collectif. Je peux répondre aussi que le div englobant n'a qu'à être conçu avec un width de largeur fixe pour que le textaera de crayons ne dépasse pas.

Donc il y a un problème à cet endroit définitivement.

Donc le mieux serait de détecter en js le width du div englobant, puisqu'après tout c'est la conception du site qui permet d'avoir un texte large, autant la respecter.

la PR est là https://git.spip.net/spip-contrib-extensions/crayons/commit/ed4c2620cf2ccc43260549724f0ec814bab5821e et voici ce que répond cerdic Salut Touti, ce max-width est là depuis toujours ou presque, pour des raisons principales de lisibilité : si on édite sur un div qui est très large on se retrouve avec des lignes de texte beaucoup trop longues. Dans la dernière modif le concernant, j'avais modifié le max-width fixe en px en le passant en rem, ce qui est plus générique https://git.spip.net/spip-contrib-extensions/crayons/commit/b61034d6c81257a7b890632d0c5a9a81d9be364e Je ne pense pas que le supprimer soit une bonne idée, c’est généralement mieux de l’avoir. Si il y a des cas précis qui ne le justifient pas je pense qu’il faut simplement personaliser dans la css de ton site, non ? -- Cédric ---------------------------- Salut Cédric, Ehhh mais non, c'est pas cool de répondre que j'ai qu'à modifier sur "mon" site, parce qu'évidemment je ne serai pas ici si l'intérêt n'était pas collectif. Je peux répondre aussi que le div englobant n'a qu'à être conçu avec un width de largeur fixe pour que le textaera de crayons ne dépasse pas. Donc il y a un problème à cet endroit définitivement. Donc le mieux serait de détecter en js le width du div englobant, puisqu'après tout c'est la conception du site qui permet d'avoir un texte large, autant la respecter.
cerdic commented 6 months ago
Owner

Je pense que c'est déjà le cas : les crayons récupèrent le width du div englobant que l'on edite et l'appliquent au formulaire.

Dans l'archeologie de la zone, j'ai exhumé le commit d'origine de ce max-width
https://zone.spip.net/trac/spip-zone/browser/spip-zone/plugins/stable/crayons/crayons.css?annotate=blame&rev=10482
qui date donc d'il y a 14 ans et disait

max-width sur le formulaire des crayons, de maniere a ce que les boutons ne soient pas completement a droite

Si vous êtes plusieurs à avoir une mauvaise expérience avec ce max-width alors il faut peut-être le supprimer en effet et voir à l'usage ce que ça donne...

Je pense que c'est déjà le cas : les crayons récupèrent le width du div englobant que l'on edite et l'appliquent au formulaire. Dans l'archeologie de la zone, j'ai exhumé le commit d'origine de ce max-width https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/_stable_/crayons/crayons.css?annotate=blame&rev=10482 qui date donc d'il y a 14 ans et disait > max-width sur le formulaire des crayons, de maniere a ce que les boutons ne soient pas completement a droite Si vous êtes plusieurs à avoir une mauvaise expérience avec ce max-width alors il faut peut-être le supprimer en effet et voir à l'usage ce que ça donne...
touti commented 6 months ago
Poster
Collaborator

En fait, c'est à partir des remontées que j'ai eu, car le rétrécissement de la largeur a gêné certain•es rédacteurices surement suite à la modification en rem il y a quelques mois b61034d6c8
J'ai juste eu à supprimer ce max-width pour que tout le monde soit content.
Donc, on peut effectivement essayer de le supprimer et attendre les retours d'expériences :)

En fait, c'est à partir des remontées que j'ai eu, car le rétrécissement de la largeur a gêné certain•es rédacteurices surement suite à la modification en rem il y a quelques mois https://git.spip.net/spip-contrib-extensions/crayons/commit/b61034d6c81257a7b890632d0c5a9a81d9be364e J'ai juste eu à supprimer ce max-width pour que tout le monde soit content. Donc, on peut effectivement essayer de le supprimer et attendre les retours d'expériences :)
cerdic commented 6 months ago
Owner

Ah peut-être parce que c'est un site avec une petite taille de police et que du coup le 44rem conduit à une largeur plus petite que les 703px initiaux ?

Ah peut-être parce que c'est un site avec une petite taille de police et que du coup le `44rem` conduit à une largeur plus petite que les `703px` initiaux ?

heu... j'ai le problème sur tous les sites où j'installe les crayons...
Je plussoie sur :

le div englobant n'a qu'à être conçu avec un width de largeur fixe pour que le textaera de crayons ne dépasse pas.

heu... j'ai le problème sur *tous* les sites où j'installe les crayons... Je plussoie sur : > le div englobant n'a qu'à être conçu avec un width de largeur fixe pour que le textaera de crayons ne dépasse pas.

pareil, il me semble que sur mes sites ça fait à peu près systématiquement un truc pas à la bonne taille

pareil, il me semble que sur mes sites ça fait à peu près systématiquement un truc pas à la bonne taille
Sign in to join this conversation.
No Label
No Milestone
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.