-
Notifications
You must be signed in to change notification settings - Fork 162
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
Améliore l'affichage des signatures #2782
Conversation
euh... ca serait cool d'en discuter un peu avant non ? Je veux dire si initialement les signatures avaient été décidé d'être limité à une ligne c'était probablement pour une raison non ? |
+1
|
Voici une petite vidéo de démonstration pour que vous puissiez avoir un aperçu avant de commenter ! :) |
Je ne commenterais plus ici tant qu'un consensus n'est pas atteint sur le forum et/ou que les discussions ne s'y sont pas éclaircies. Là j'ai juste le sentiment de la proposition d'une solution sans savoir ce que sont les autres. |
Tout pareil que les autres, je ne sais pas trop d'où débarque cette PR. Le visuel final est plutôt propre mais je ne suis pas certain de vouloir cette fonctionnalité. La signature n'est pas faite pour écrire sa bio (il y a un espace pour ça) mais juste de donner 2-3 informations si on veut. De plus, les utilisateurs sans JS activé vont avoir des gros pavés et je me demande si ça ne sera pas encore pire pour les malvoyants (niveau accessibilité, etc.). |
Sans JS ça reste comme actuellement. Pour ce qui est de l'accessibilité, si les signatures sont plus longues ils auront des signatures plus longues (mais ce n'a rien à voir avec le JS). |
(Faudra reparer les tests qui vérifie la longueur dans le formulaire) |
Je sais mais j'attends qu'on se mett e d'accord sur une longueur ! |
Bah on pourra partir du principe que "2000 caractères c'est la taille maximale que l'on accepte pour une URL" |
Travis signale une erreur ici car il faut mettre à jour ce TU pour créer une chaine de plus de 2000 caractères |
91b5a97
to
598a2f4
Compare
J'ai corrigé le TU ! |
Les liens sont-ils "tronqué" après tant de caractère comme dans les Tweet sur Twitter ? Sinon pourquoi ne pas le faire ? |
Non car tout simplement on a aucun moyen technique pour le faire ^^ |
Je suis un utilisateur de php et très peu de python, mais je ne vois pas trop qu'est-ce qui fais que c'est impossible ... |
@WinXaito : On pourrait éventuellement passer par un raccourcisseur d'url comme sur Twitter mais ça ne changerait rien au problème, il faudra toujours mettre l'url complète dans le formulaire ! |
Non, on ne pourrait pas. Les raccourcisseurs d'URL sont une plaie qui doit être éliminée du web. |
Je ne comprend pas pourquoi ? Je trouve ceci pourtant bien pratique quand l'url se trouve dans un texte et que nous ne souhaitons pas être déranger... Après idéalement mettre en infobulle l'url complète
On recherche s'îl y a une url dynamiquement, et si on en détecte une, alors on fixe un certain nombre de caractère utilisé ? Bref mon but n'est bien sur pas de lancer un débat la dessus, par contre je serais ravi d'avoir l'avis de @SpaceFox concernant ce qu'il a dit. |
J'ai pas dit que c'est impossible j'ai dit qu'on a pas actuellement de moyen technique pour le faire. On a pas la base de code ni rien si tu preferes |
@WinXaito : Je pense que pour l'instant le mieux est de rester sur l'augmentation à 2000 caractères. Je pense franchement que ça va suffire. On pourrait surement calculer exactement le nombre de caractères que la signature prend, mais ce serait overkill pour une signature (qui n'est pas la fonctionnalité majeure du site) ! |
QA : NOK.
Information de mon système : OS X 10.9.5 et Chrome 46.0.2490.71 (64-bit) |
Ca va beaucoup mieux quand on génère le front (pas l'habitude ..). Nouvelle QA :
|
598a2f4
to
aaf792c
Compare
Voilà j'ai pushé le fichier de migrations. Pour ce qui est de la taille : rien ne t'oblige à cliquer et on peut un peu faire confiance à la communauté et modérer au besoin ? |
Par contre, des actions sur les +/-1 font bouger la signature. C'est pas top.
Faire confiance à l'utilisateur ? Ca semble être une si mauvaise idée ! :) Moi je voudrais simplement un dernier feu vert de @Eskimon suite à ma capture. |
ah la vache, je me rendais pas compte de ce que ca faisais 2000 caractères, ca pousse très fortement à la réflexion pour le coup... |
Du coup, je met un veto sur ma propre décision, ca mérite franchement d'être réfléchi mieux que ca car là on s'ouvre à tout et n'importe quoi. On ne doit pas pouvoir écrire un roman dans une signature qui est censé représenté un trait d'esprit ou une ouverture sur des liens, mais pas un roman ou une présentation de projet entière ! |
750 caractères devraient suffirent pour faire une ou deux phrases avec des liens Markdown dedans (500 risque d'être un peu juste, non ?) |
Ben fait un test et poste un ou deux screen, qu'on juge ;) |
J'ai testé avec 500 caractères (contenant 3 liens et un peu de mise en forme) :
Voilà le résultat : Je pense que c'est un assez bon compromis entre 255 et 2000 car ça permet de mettre quelques liens sans trop abuser non plus. Par ailleurs, je précise que le champ pour entrer la signature reste sur une ligne donc ça va refroidir d'écrire beaucoup. |
Franchement, c'est pas du tout gênant ! Vous en pensez quoi les autres ? |
Que ca fera un nouveau ticket dans "Les petits pixels" :D Tu aurais une capture d'écran avec uniquement du texte, sans lien ? |
:D si. C'est le but même de ce sujet ^^ Bon, c'est déjà moins pire avec 500... |
Voilà ce que ça donne https://drive.google.com/file/d/0ByCc7U2bgTkbRHJKTENhSDZoMEE/view?usp=sharing Je pense pas que les gens vont croire que c'est un bug. Et puis s'ils croient que s'en est un, bah on leur dit que non. C'est pas grave d'avoir un truc qui bouge. |
aaf792c
to
9c76622
Compare
Bon, QA ok, ici. ON y va comme ça ? |
J'ai envie de dire "mettons cela à l'essai, on pourra toujours revert facilement la PR si vraiment ca va pas du tout" (mais si je devais m'en tenir à mon avis perso et pas celui de CdP : Je trouve pas ca terrible non plus, un truc me chagrine sans que je sache quoi...) |
Bon, je merge, on verra bien l'accueil du public... |
Améliore l'affichage des signatures
Augmente la longueur autorisée des signatures (pour que les utilisateurs puissent mettre des liens avec des longues URLs)
Affiche la signature complète lorsque l'on clique dessus (et la réduit si l'on re-clique dessus)
Voici une petite vidéo de démonstration !
QA :
python manage.py migrate
;npm run gulp -- build
;