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

Manque de cohérence dans l'affichage du bouton d'aide markdown #1691

Closed
firm1 opened this issue Nov 6, 2014 · 11 comments
Closed

Manque de cohérence dans l'affichage du bouton d'aide markdown #1691

firm1 opened this issue Nov 6, 2014 · 11 comments
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Milestone

Comments

@firm1
Copy link
Contributor

firm1 commented Nov 6, 2014

Firefox 33
Windows 7

Cf. Image. Dans la première capture, on voit que le bouton "Afficher l'aide markdown" n'est pas sur la même ligne que les boutons "Aperçu" et "Envoyer"

Le mauvais exemple :

eloigne

Le bon exemple :

gud

@firm1 firm1 added S-BUG Corrige un problème C-Front Concerne l'interface du site labels Nov 6, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Nov 6, 2014

Très honnêtement, ça fait un nombre de fois incalculable que c'est remonté et que je réponds à chaque fois : #465 et #464

C'est pas nouveau, et on est pas magicien.

Résumé :

Le formulaire de réponse à un sujet est soigné : je l'ai recodé de A à Z en shintant la gestion des formulaire du back, puisque non adapté.

Sur les formulaires des tutos, édition de message, etc. J'ai laissé le back faire car trop compliqué de réinventé la roue quand il y a de la gestion de validation de formulaire.

La solution est simple et je la demande depuis perpette : faire un template pack, notamment pour avoir un nouveau type de champ personnalisé pour l'éditeur markdown (plutôt que de bidouiller avec des classes à chaque fois, on aurait un type de champ précis dans le back). Aussi, le bouton et l'aide passeraient dans ce template pack plutôt que de fix dégueu en JS.

@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Nov 6, 2014
@firm1
Copy link
Contributor Author

firm1 commented Nov 7, 2014

Sur mobile c'est pire. Le bouton d'aide est caché derrière les boutons aperçu et envoyer. Cf. Screen.

screenshot_2014-11-07-17-49-52

@Alex-D
Copy link
Contributor

Alex-D commented Nov 8, 2014

Y la PR #1558 qui a corrigé ça. Vérifie ton cache ou alors il y a un soucis.

@firm1
Copy link
Contributor Author

firm1 commented Nov 8, 2014

Il y'a bien un souci. Cache verifié

@Alex-D
Copy link
Contributor

Alex-D commented Nov 10, 2014

J'ai en effet le même rendu :/ Pourtant la PR visait à corriger tout ça... je regarderais.

@Eskimon
Copy link
Contributor

Eskimon commented Apr 21, 2015

Ca existe encore ce problème ?

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

firm1 commented Apr 21, 2015

bug toujours présent

@gustavi
Copy link
Contributor

gustavi commented Dec 5, 2015

Alors j'ai identifié le bug mais je ne sais pas le corriger.

Ça vient du fichier https://github.com/zestedesavoir/zds-site/blob/dev/assets/js/markdown-help.js.

En gros ça ajoute le bouton après .md-editor sauf que cette classe ne se trouve pas partout au même endroit.

Sur la création d'un topic ça vient sur le textarea sauf que celui-ci est encapsulé dans d'autres <div> donc control-groups. On a donc un comportement foireux.

Sur l'envoi d'un message c'est correct puisque le textarea est "seul".

Si quelqu'un a une idée je suis preneur.

@Situphen
Copy link
Member

Situphen commented Dec 5, 2015

Comme l'a dit Alex-D plus haut :

La solution est simple et je la demande depuis perpette : faire un template pack, notamment pour avoir un nouveau type de champ personnalisé pour l'éditeur markdown (plutôt que de bidouiller avec des classes à chaque fois, on aurait un type de champ précis dans le back).

Actuellement on utilise django-crispy-forms donc on est complètement dépendant des choix de leurs templates. Si on veut changer des trucs (pour résoudre ce problème par exemple), il faut faire un fork.

@gustavi
Copy link
Contributor

gustavi commented Dec 5, 2015

Je pense vraiment qu'on peut se passer de faire un fork avec un peu de bidouille. Il suffit de mettre l'attribut .md-editor au bon endroit, quitte à le renommer.

@Situphen
Copy link
Member

Situphen commented Dec 5, 2015

Je te laisse essayer... Si tu y arrives sans foirer et sans que ce soit trop crade, GG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

6 participants