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

Mise à jour de la toolbar de l'éditeur #3021

Closed
Thunderseb opened this issue Sep 29, 2015 · 11 comments
Closed

Mise à jour de la toolbar de l'éditeur #3021

Thunderseb opened this issue Sep 29, 2015 · 11 comments
Labels
C-Front Concerne l'interface du site
Milestone

Comments

@Thunderseb
Copy link
Contributor

Coucou,

J'ai fait une mise à jour du script editor.js ( https://github.com/zestedesavoir/zds-site/blob/dev/assets/js/editor.js ). Voici ce qu'il y a de neuf :

  • ajout d'un bouton pour insérer un code inline
  • ajout d'un bouton + sous-menu avec les smileys
  • ajout d'un bouton + sous-menu avec des caractères spéciaux (du genre guillemets français et majuscules accentuées)
  • pour les boites de dialogue image et lien, les champs ont le focus
  • ajout d'une class CSS "zform-button-useless" sur certains boutons moins utiles dans l'optique des les masquer avec des media-queries CSS pour rendre le truc un peu plus responsive

Comme y'a deux nouveaux boutons, y'a deux nouvelles images à ajouter dans les CSS (fichier + icônes donné).

Vlà le zip avec les fichiers à intégrer :

http://s.sdlm.be/zestedesavoir/editor.js-29-09-2015.zip

@Situphen Situphen added Evolution C-Front Concerne l'interface du site labels Sep 29, 2015
@Situphen
Copy link
Member

Cool, j'essaye de regarder ça ce soir ! :)

@Thunderseb
Copy link
Contributor Author

Btw, je ferai une autre màj car je compte intégrer, dans la modale d'insertion d'image, un moyen de parcourir la galerie.

Je prévois aussi la possibilité d'afficher/masquer certains boutons et de changer leur ordre. Mais ce sera pour après. Le code est déjà prévu pour (l'ordre est défini par défaut dans zForm.buttons).

@Situphen
Copy link
Member

En gros tu comptes développer la ZEP sur l'éditeur ? :)

@Situphen
Copy link
Member

J'ai transposé le code de ton archive dans celui de ZdS, ce qui donne ça.

J'ai fait :

  • copier/coller du code JS de l'éditeur
  • copier/coller du CSS dans le fichier SCSS (les deux nouveaux icônes)
  • création d'une règle pour cacher les boutons qui ont la classe zform-button-useless

Je me suis permis de faire quelques petits changements :

  • suppression des espaces inutiles (ligne vide ou fin de ligne)
  • transformation de quelques tabulations perdues dans le code en espaces
  • traduction de certains commentaires en anglais (je ferais les derniers plus tard)

Par ailleurs, j'ai vu d'après ton code que tu supportes IE ? ZdS ne support pas IE donc ce n'est pas la peine de te casser la tête. (Je n'ai néanmoins pas supprimé le code actuel spécial IE car je sais que ça représente du boulot en plus.)

Peux-tu écrire un message dans le sujet de la ZEP sur l'éditeur pour parler des modifications que tu as codé et que tu comptes coder ? Notamment des boutons a cacher sur mobile qui, je pense, méritent un débat pour savoir lesquels cacher ! :)

@DevHugo
Copy link
Contributor

DevHugo commented Oct 1, 2015

Merci, j'attend ces nouvelles fonctionnalisées avec impatience ! Surtout pour les images et la gallerie !

@GerardPaligot
Copy link
Member

En gros tu comptes développer la ZEP sur l'éditeur ? :)

Je connais pas la ZEP sur l'éditeur mais si la ZEP se contentait de rajouter des boutons dans l'éditeur actuel, je serais déçu. :)

@DevHugo
Copy link
Contributor

DevHugo commented Oct 1, 2015

ça allait plus loin, de mémoire, c'était, upload d'image automatique (déplacer dans le textarea), à la GitHub quoi, les packs de smileys, et d'autres trucs que j'ai oublié.

@artragis
Copy link
Member

artragis commented Dec 9, 2015

dis @sandhose pourquoi la PR n'est pas mise en place?

@artragis
Copy link
Member

artragis commented Dec 9, 2015

heu c'était @Situphen

@Situphen
Copy link
Member

Situphen commented Dec 9, 2015

Euh, très bonne question !

@SpaceFox
Copy link
Contributor

SpaceFox commented Mar 5, 2016

Hop, c'est mergé !

@SpaceFox SpaceFox closed this as completed Mar 5, 2016
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
Projects
None yet
Development

No branches or pull requests

6 participants