yeswiki icon indicating copy to clipboard operation
yeswiki copied to clipboard

fix(action-bilder/flying-bar): display at right place for textarea

Open J9rem opened this issue 3 years ago • 6 comments

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

J9rem avatar Mar 21 '21 18:03 J9rem

@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 : 2021-03-21 01-48 - Capture d'écran

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.

J9rem avatar Mar 21 '21 18:03 J9rem

@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 avatar Jul 16 '21 15:07 J9rem

@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.

acheype avatar Jul 16 '21 15:07 acheype

OK, je reprends donc la PR semaine prochaine pour la retester avant de passer en revue.

J9rem avatar Jul 16 '21 16:07 J9rem

any news on this PR 😉 ?

mrflos avatar Sep 04 '21 11:09 mrflos

Je clos cette issue car il y a toujours la possibilité de cliquer sur 'composants' > 'éditer ce composant' en alternative

J9rem avatar Oct 01 '21 08:10 J9rem

Coucou !

Je ferme cette PR puisque la refactor d'aceditor fix le probleme

seballot avatar Oct 10 '22 05:10 seballot