Améliorations diverses d'Etherpad

Cette MR inclut un bump de la version (1.7.0 → 1.7.5), des changements de plugin et de skin. Elle ajoute également un README qui aide à faire évoluer l'image.

Changements au niveau de l'image :

• Les paramètres importants peuvent être modifiés via les variables d'environnement du conteneur. Voir le README (disponible : minification, titre, port, skin)
• Ajout du package abiword, nécessaire pour les exports PDF/ODT.
• Modification des packages npm pour les plugins : voir plus bas

Configuration :

• Supporte la minification des CSS|JS par la variable ETHERPAD_MINIFY (faux par défaut)
• Empêche l'import de fichiers dont l'analyse n'est pas supportée
• Langue par défaut en français (nécessite de supprimer les cookies)
• unknown -> anonyme pour les auteurs inconnus

Esthétique :

• Passage au skin Colibris (défaut dans la future 2.0)
• Changement du nom de l'instance
• Message plus informatif lors de l'ouverture d'un nouveau pad

Plugins :

• Ajout de chatdate (horodatage des messages du chat)
• Ajout de font_family (choix de la police)
• Ajout de hide_referrer (pour éviter de divulguer l'adresse du pad lors du clic sur un lien)
• Ajout de pad_activity_notification_in_title : ajoute un caractère * dans le titre du pad s'il a été modifié pendant que l'on était pas sur l'onglet
• Ajout de set_title_on_pad : permet de nommer un pad et n'oblige pas à rajouter un titre H1
• Ajout de tables2 : support des tableaux
• Suppression de spellcheck (inutile)
• Suppression de markdown (peu utilisé et dysfonctionnel)
• Suppression de table_of_contents (incompatible avec le nouveau skin)

Dirty fixs :

• Augmentation de la largeur de la zone d'édition (à coup de sed approximatif dans le Dockerfile + ajout d'une règle CSS !important, c'est dire si c'est sale)
• Changement du titre par défaut, codé en dur dans le plugin concerné, également à coup de sed dans le Dockerfie

changed the description

added 1 commit

• 41a2d41a - Fix welcome message

Compare with previous version

added 1 commit

• 4cce1e28 - Fix welcome message

Compare with previous version

Sur les instances de Framasoft, les options « Page en largeur pleine » et « Afficher la table des matières ». La première me semble assez importante pour la prise de note, la seconde peut être pratique pour montrer qu'on peut faire des vrais trucs structurés avec. Je pense que ça serait bien de rajouter ces deux options si c'est possible.

Note à moi même :

• https://github.com/JohnMcLear/ep_prompt_for_name ce plugin est cool
• Concernant la full page : Je pense que c'est pas natif, Framasoft a du rajouter ça "à la main". Trouvé dans la source d'un pad random :

<script type="text/javascript">
    function toggleColibriFullWidth() {
        var $outerIframeContents = $('iframe[name="ace_outer"]').contents();
        var $innerIframe         = $outerIframeContents.find('iframe');
        var $innerdocbody        = $innerIframe.contents().find("#innerdocbody");
        if ($('#options-colibris_full_width').is(':checked')) {
            $outerIframeContents.find('#outerdocbody').addClass('full-width');
        } else {
            $outerIframeContents.find('#outerdocbody').removeClass('full-width');
        }
    }
</script>

Et impossible de trouver une référence à cette fonction sur les moteurs de recherche, sur les dépôts, etc... Donc probablement à injecter à la main, à moins que quelqu'un ne trouve d'où ça sort.

@uroremi @huetremy Tiens d'ailleurs, sur Framapad, la pleine page casse les commentaires (leur affichage du moins). Est-ce-que ça vaut le coup ?

Autre problème que je trouve assez important si on veut le présenter comme un outil pour faire des brouillons et pas seulement de la prise de note : Les exports sont pas bons : Les couleurs et les titres passent pas à l'exportation, les autre polices rendent l'exportation impossible, et les tableau cassent l'ODT et rendent mal en PDF et en doc.

added 3 commits

• 1616bca0 - Add plugin to set Colibri skin full page
• 40d1784d - Remove special separator char from welcome message
• 7db5d261 - Switch from abiword to unoconv for better import/export

Compare with previous version

added 1 commit

• b983648c - Remove unoconv setting

Compare with previous version

added 2 commits

• 5e638bdc - Add ep_prompt_for_name to encourage users to use a pseudo/name after 5s
• eeb473b4 - Add path to soffice so the unoconv export/import works

Compare with previous version

added 1 commit

• 7178d24e - Add path to soffice so the unoconv export/import works

Compare with previous version

Bon, la plupart des trucs sont ok, manque juste de trouver pourquoi les exports unoconv font crasher Etherpad

etherpad-app          | [2019-05-06 23:37:05.028] [ERROR] console - TypeError: callback is not a function
etherpad-app          |     at /opt/etherpad-lite/src/node/utils/ExportHtml.js:483:9
etherpad-app          |     at /opt/etherpad-lite/src/node/utils/ExportHtml.js:66:5
etherpad-app          |     at /opt/etherpad-lite/src/node/utils/ExportHtml.js:58:5
etherpad-app          |     at fn (/opt/etherpad-lite/src/node_modules/async/lib/async.js:641:34)
etherpad-app          |     at Immediate.<anonymous> (/opt/etherpad-lite/src/node_modules/async/lib/async.js:557:34)
etherpad-app          |     at runCallback (timers.js:705:18)
etherpad-app          |     at tryOnImmediate (timers.js:676:5)
etherpad-app          |     at processImmediate (timers.js:658:5)
etherpad-app          |     at process.topLevelDomainCallback (domain.js:120:23)

J'avais eu le même genre de soucis avec abiword, ça venait du fait que le path était pas dans la config, mais là j'ai bien mis "soffice": "/usr/bin/soffice"... À suivre...

added 1 commit

• 6431be94 - Remove ep_unoconv as support for LibreOffice is native in Etherpad lite

Compare with previous version

Trouvé, ep_unoconv est obsolète, le support est natif. J'ai bien les couleurs en HTML, par contre pas en ODT ni PDF, ni les titres. @uroremi encore autre chose ?

Peut-être un rapport avec :

etherpad-app          | [2019-05-06 23:59:14.626] [INFO] console - req.params.pad nouveau_pad
etherpad-app          | [2019-05-06 23:59:14.626] [INFO] console - Found  null  for  nouveau_pad

Lors de l'export, à creuser. Semblerait qu'il n'y ait rien de particulier à rajouter pourtant.

added 1 commit

• 4c03380f - Re-add markdown, useful for export and editing, remove dirty CSS fix

Compare with previous version

resolved all discussions

@huetremy Je pense que le merge est prêt, néanmoins j'aimerais vérifier que tout est puisque c'est un peu le bordel. Tu peux aussi double check sur l'instance de test pour être sûr.

merged

mentioned in commit 9ef4bbe0