- Issue created by @breidert
- ๐ฉ๐ชGermany breidert
The code in the MR works nicely. You get the same btton in a baloon menu.
The result looks like this:
- ๐ฌ๐งUnited Kingdom MrDaleSmith
That works very nicely and is a good idea. I can't quite shake the suspicion that this is a little non-standard, as a lot of the other buttons on the CKEditor toolbar *could* also work the same way (formatting, font choices) but don't. I'll let the maintainers have the final decision, but at the very least it might be nice to make this something you could turn off in a setting?
- ๐ต๐นPortugal bbruno Poland
This is a nice idea, I have tested it locally and it works as expected. I agree with mrdalesmith that it perhaps is a little non-standard, but in my opinion provides a much better ux than the current option.
Non-related to the issue itself, but the changes in the package-lock.json and yarn.lock indicate that we have multiple package handlers at work here, it would be best to stick to one to avoid any inconsistencies due to unsynced lock files.
- ๐ฉ๐ชGermany a.dmitriiev
a.dmitriiev โ made their first commit to this issueโs fork.
- ๐ฉ๐ชGermany a.dmitriiev
I have split main plugin and balloon plugin in the way, that if you want the balloon menu you need to drag a new item to toolbar. This item is invisible. If you don't need it, the behavior will still be the same.
- ๐ฎ๐ณIndia snehal-chibde
Hello, I have tested this locally and it works as expected.
Tested on Drupal 11.1.6 version, AI Module version1.0.x.
Baloon menu is visible as expectedAdded a before and after screenshot of the same.
- ๐ฎ๐ณIndia prashant.c Dharamshala
This is an extremely useful feature to have. It is working nicely; however, one minor observation is that sometimes when there is text below the selected text, then the positioning of the button "AI assistant" is a bit off, however working properly.
- ๐ฉ๐ชGermany marcus_johansson
Hi everyone, awesome function, code reviewed and merged