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

Allonger le nombre de caractères limites dans la signature des forums #1393

Closed
ghost opened this issue Aug 24, 2014 · 17 comments
Closed

Allonger le nombre de caractères limites dans la signature des forums #1393

ghost opened this issue Aug 24, 2014 · 17 comments
Assignees
Labels
C-Back Concerne le back-end Django
Milestone

Comments

@ghost
Copy link

ghost commented Aug 24, 2014

Salut !

Petite suggestion : allonger le nombre de caractères max pour rédiger sa signature.

En effet quand on veut mettre un lien ou deux, ça prend vite de la place...

Iz it possibeul ?

Merci d'avance, et pour votre boulot sur le site ;)

@Axylium
Copy link
Contributor

Axylium commented Aug 24, 2014

Il y a eu un débat sur le site à ce propos très récemment, si tu ne l'aurais pas vu, il a un peu dériver, mais bon.

Sinon, pour te répondre, il faudrait voir avec Alex, je pense. Donc ping @Alex-D. ^^

@ghost
Copy link
Author

ghost commented Aug 24, 2014

Oké merci ;)

@Axylium
Copy link
Contributor

Axylium commented Aug 30, 2014

J'en profite pour en parler là, vu que ça traite des signatures. Quand nous avons une signature longue et qu'on est auteur du sujet, et que donc que nous avons : « cette réponse m'a aidé », cette dernière phrase, vient s'apposer sur le dessus de la signature et empêche de ce fait de voir le reste.

capture du 2014-08-30 18 29 16

@Alex-D
Copy link
Contributor

Alex-D commented Aug 31, 2014

C'est voulu. Le site est fluide, c'est impossible de garantir que la signature sera complètement affichée correctement sur tous les supports.

@Alex-D
Copy link
Contributor

Alex-D commented Sep 4, 2014

Du coup, je considère ce ticket comme résolu, car tout ce que vous me dites est voulu, pensé et réfléchi (bien que discutable, mais il faut bien faire des choix à un moment).

@Alex-D Alex-D closed this as completed Sep 4, 2014
@SpaceFox SpaceFox added the Back label Sep 4, 2014
@SpaceFox SpaceFox added this to the Version v1.x milestone Sep 4, 2014
@SpaceFox
Copy link
Contributor

SpaceFox commented Sep 4, 2014

Pour moi le problème n'est pas résolu : le sujet d'origine, c'est la longueur du champ technique qui empêche de mettre plusieurs liens si le href des liens est trop long.

La taille d'affichage dont parle @Florianboux est à moitié un HS - mais je suis d'accord, ce point est résolu.

@SpaceFox SpaceFox reopened this Sep 4, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Sep 4, 2014

Exact, j'avais mal interprêté en relisant, je n'avais pas le bon point de vue.

Du coup, c'est un soucis purement back et une solution qui avait été énoncée est de :

  • mettre une longueur de champ assez longue, à la fois à l'input et au champ en BDD
  • parser la chaîne de MD à HTML, puis faire un striptags et tester si cette longueur est inférieure à celle que l'on souhaite

@firm1
Copy link
Contributor

firm1 commented Sep 5, 2014

Pour information, pour résoudre cette issue, il faudrait

  • faire sauter la limite des 250 caractères en modifiant cette ligne par ça :
sign = models.TextField('Signature', max_length=1000, blank=True)
  • lancer la génération des fichiers de migrations (en ligne de commande) :
python manage.py schemamigration member --auto

Et ça devrait faire une jolie PR !

@Eskimon
Copy link
Contributor

Eskimon commented Sep 5, 2014

C'est pas suffisant, il faudra tout de meme limite en longueur la signature "rendu en MD" si on veut pas que des petits malins mettent des signatures qui font des kilometres mais que l'on affiche pas.

@firm1
Copy link
Contributor

firm1 commented Sep 5, 2014

si on veut pas que des petits malins mettent des signatures qui font des kilometres mais que l'on affiche pas.

s'il a un écrans quifait des kilomètres à l'instar de sa signature, ça sera affiché :)

On ne peut pas juger de la limite au poil près (ça dépend du support), donc si le mec met un message super long, tout dépend de son écran.

@Eskimon
Copy link
Contributor

Eskimon commented Sep 5, 2014

Et le mec qui te met l'integrale des miserables dans sa signature ?

@firm1
Copy link
Contributor

firm1 commented Sep 5, 2014

Et le mec qui te met l'integrale des miserables dans sa signature ?

En effet :). J'édite mon message donc.

@Eskimon Eskimon removed this from the "Futur proche" (v1.x) milestone Apr 21, 2015
@SpaceFox
Copy link
Contributor

SpaceFox commented Oct 1, 2015

C'est toujours d'actualité @Situphen ?

@Situphen
Copy link
Member

Situphen commented Oct 1, 2015

Bah c'est le débat communautaire qui coince !

Tout le monde semble d'accord pour agrandir le nombre de caractères dans les signatures. Mais ça coince sur est-ce qu'on ajoute la possibilité de cliquer sur la signature pour voir la signature complète (une vidéo est disponible ici) ? Des gens pour, d'autres contre. Dans les derniers messages, il me semble avoir un peu un consensus pour tester et voir si les gens abusent du truc ou pas.

Je pense que le mieux est que tu discutes ça avec Eskimon et que vous décidiez si oui ou non on teste ça ! En tout cas, ça sert à rien de débattre plus longtemps pour un truc aussi mineur que ça... :)

@Eskimon
Copy link
Contributor

Eskimon commented Oct 13, 2015

Bon, je vais dans ma grande sagesse trancher le souci : On pousse à 2000 et on autorise l'affichage complet au clic sur la signature (pas de nouveaux boutons et attention à ne pas empêcher les clics sur URL)

@SpaceFox
Copy link
Contributor

Je plusseoie la solution de @Eskimon .

@Situphen
Copy link
Member

Bah normalement ma PR #2782 respecte ça donc on peut la QAtiser !

@Eskimon Eskimon added this to the Version de développement milestone Nov 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
None yet
Development

No branches or pull requests

6 participants