-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Balloon toolbar overflows outside the editor #17814
Comments
Please always add screenshots to the issues. Thanks to that, we will be able to quickly asses if it's an issue related to some other or a duplicate. |
Sorry, I don't know why I didn't add a screencast. Done already 👌🏼 |
If you mean the balloon editor, from what I see the toolbar can also float outside the editor (I don't know if it matters, but for me it was more reproducible and noticeable at slower scrolling). Screen.Recording.2025-01-28.at.13.22.14.movIt also looks like a problem from that former PR - #14755 |
All screencasts show balloon toolbar, PR related to features' balloon toolbar. @pszczesniak I assume there's a difference. There's also |
Yes, it needs a deeper investigation. For "simple" balloon it works ok - i've tested it here (i've added height to the editing view div). When i want to add a link on a selected word and scroll it outside editor - it disappeared, like it should. In some cases it's not working with toolbars balloons. But image and table toolbars works ok (tested on full-feature example with editor height set to 300px). |
📝 Provide detailed reproduction steps (if any)
(This is a classic editor sample generated from the builder with a balloon toolbar, only the editor height has been changed and a header has been added)
✔️ Expected result
The balloon toolbar remains within the boundaries of the editor and does not overflow outside of it.
❌ Actual result
The balloon toolbar extends beyond the boundaries of the editor and overflows outside of it.
overflowing.mov
📃 Other details
If you'd like to see this fixed sooner, add a 👍 reaction to this post.
The text was updated successfully, but these errors were encountered: