Skip to content

The image toolbar shouldn't be attached to the link #900

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

Closed
Mgsy opened this issue Mar 14, 2018 · 0 comments Β· Fixed by ckeditor/ckeditor5-ui#406
Closed

The image toolbar shouldn't be attached to the link #900

Mgsy opened this issue Mar 14, 2018 · 0 comments Β· Fixed by ckeditor/ckeditor5-ui#406
Labels
type:bug This issue reports a buggy (incorrect) behavior.
Milestone

Comments

@Mgsy
Copy link
Member

Mgsy commented Mar 14, 2018

Is this a bug report or feature request? (choose one)

🐞 Bug report

πŸ’» Version of CKEditor

Latest master.

πŸ“‹ Steps to reproduce

  1. Create some link.
  2. Open the link balloon.
  3. Focus the image.

βœ… Expected result

The image toolbar appears above the image.

❎ Actual result

The image toolbar is attached to the link.

πŸ“ƒ Other details that might be useful

GIF

bug_cke5

Here is the first bad revision - b5083b4

OS: All platforms
Browser: All browsers

@Mgsy Mgsy added the type:bug This issue reports a buggy (incorrect) behavior. label Mar 14, 2018
@Reinmar Reinmar added this to the iteration 15 milestone Mar 14, 2018
@Reinmar Reinmar modified the milestones: iteration 15, backlog Mar 30, 2018
jodator added a commit to ckeditor/ckeditor5-ui that referenced this issue Jun 8, 2018
Other: Made the `ContextualBalloon` always use the position of the topmost view in the stack (see: ckeditor/ckeditor5-table#28). Closes ckeditor/ckeditor5#900.
@Reinmar Reinmar modified the milestones: backlog, iteration 18 Jun 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
2 participants