yeswiki
yeswiki copied to clipboard
fix(action-bilder/flying-bar): display at right place for textarea
L'idée est de pouvoir utiliser le bouton avec le crayon qui permet de modifier l'action sélectionnée avec l'action-builder, lors de la modification d'une page ET lors de la modification d'une fiche qui possède un textarea.wiki-textarea avec le bon nom.
J'ouvre une PR car il y a des besoins d'échanges préalables avant déploiement avec @acheype
@acheype a écrit
Oui c'est bien ça pour le crayon, il s'affiche bien mais chez moi il est décalé, même en enlevant mon custom css :
Et j'ai testé en iframe car j'utilise ce mode en prod et je me suis rendu compte qu'on ne voit plus le bouton vu qu'il n'y a plus de marge. Mais ça le fait même à l'édition d'une page classique. Je propose du coup de rajouter pour l'iframe une marge systématique à l'édition de la largeur du bouton (à gauche et à droite à la fois pour que le contenu reste centré). Qu'en dis-tu ?
Je suis entièrement d'accord qui si la fonctionnalité du bouton est déployée, il faut faire attention au design. Toutefois, je ne trouve pas trop prioritaire de faire tous les ajustements. Pouvons-nous retarder de plusieurs semaines le déploiement de cette branche ?
Effectivement, le bouton N'A PAS été développé pour le mode iframe. Je me doute que ce soit plus pratique, toutefois, le mode iframe n'est pas selon moi une bonne utilisation générale des YesWiki. Donc, @acheype, si je comprends bien, tu proposes de faire des modifications du coeur YesWiki pour une utilisation marginale du mode iframe.
Donc, pour ce qui concerne le mode editiframe, ... je te laisse trouver dans ton custom css ce qui va bien pour ensuite faire une proposition d'ajout au bon fichier css d'une classe qui ne s'active qu'en mode iframe.
Pour le mode normal, j'ai aussi le décalage. Je vais regarder comment bien replacé le crayon, mais d'ici quelques semaines.
@acheype est-ce que tu as trouvé ce qui fonctionne en custom CSS pour pouvoir afficher le crayon de action-buider au bon endroit en mode iframe ? si oui, je veux bien que tu fasses un commit sur cette branche pour que je puisse tester et passer cette PR en revue. Merci
@J9rem ah non, j'avais zappé cette histoire de css et je t'avoue que je n'aurai pas le temps de regarder cela de si tôt. Après vas-y, ton correctif me semble plus important que cette histoire d'editiframe. Il y aura toujours moyen plus tard de définir de petites marges autour du textarea en mode iframe ou de mettre le bouton côté intérieur dans ce mode.
OK, je reprends donc la PR semaine prochaine pour la retester avant de passer en revue.
any news on this PR 😉 ?
Je clos cette issue car il y a toujours la possibilité de cliquer sur 'composants' > 'éditer ce composant' en alternative
Coucou !
Je ferme cette PR puisque la refactor d'aceditor fix le probleme