-
-
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
[Safari] Link balloon is shown outside the view in manual tests if selection is collapsed #4748
Comments
It can be also reproduced in Chrome. |
@pomek I'll update the title. |
Looks like a problem with getting offset of collapsed selection. |
@oskarwrobel Yep, I can't reproduce it either. @pomek, can you provide steps to reproduce it, along with version of Chrome, type of screen (retina, non-retina), zoom factor, and all that stuff? |
Eh. Yesterday I could reproduce it. Today I cannot. Maybe it was Firefox instead of Chrome. I am not sure. I would like to apologize for the mess. |
There's a ticket on review which fixes some cases in positioning – maybe that it will fix this issue. |
I'm afraid it is a different issue. |
Right... empty selection. I remember dealing with this in the past. It wasn't easy – I had to extend the range in some direction – otherwise the box was calculated incorrectly. And there were edge cases like empty blocks where the position had to be taken from the element. |
Here is a fix for this: ckeditor/ckeditor5-ui-default#91 |
Still as much reproducible as in the past. Could we fix it by positioning to the entire block if 0,0 cords were returned? A full fix would be to inject an empty element inline in the place of the caret and positioning according to it. This may be required later on when we'll deal with scrolling to the caret on Enter, Delete, etc. but this is much more work. |
ckeditor/ckeditor5-utils#154 should fix the issue. |
Test: http://localhost:1030/build/modules/amd/tests/link/manual/link.
When I click on the first word in editor, the link balloon is shown as below:
The text was updated successfully, but these errors were encountered: