Skip to content
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

Editor toolbar Outside parent div. #14794

Closed
compjotr opened this issue Aug 15, 2023 · 4 comments
Closed

Editor toolbar Outside parent div. #14794

compjotr opened this issue Aug 15, 2023 · 4 comments
Labels
package:table package:ui resolution:resolved This issue was already resolved (e.g. by another ticket). type:bug This issue reports a buggy (incorrect) behavior.

Comments

@compjotr
Copy link

πŸ“ Provide detailed reproduction steps (if any)

  1. add a table.
    2.Scroll down so the editor toolbar is above the screen
  2. highlight a table cell.

βœ”οΈ Expected result

the toolbar editor dosen't move outside parent.

❌ Actual result

toolbar is outside parent. It shouldn't be visible at this point.

❓ Possible solution

it is possible to edit the values here: .ck.ck-toolbar>.ck-toolbar__items {
display: flex;
flex-flow: row wrap; REMOVE THIS LINE
align-items: center;
flex-grow: 1; REMOVE THIS LINE }
but then the items in the toolbar overflows.

πŸ“ƒ Other details

Screenshot 2023-08-15 at 16 18 21

  • Browser: Chrome
  • OS: MacOS monterey, Windows 10
  • First affected CKEditor version: …
  • Installed CKEditor plugins:

@compjotr compjotr added the type:bug This issue reports a buggy (incorrect) behavior. label Aug 15, 2023
@Witoso
Copy link
Member

Witoso commented Aug 16, 2023

Could you create some recording? Just by looking at the picture it's hard to tell what's wrong as it shows correct behavior.

@Witoso Witoso added package:ui package:table pending:feedback This issue is blocked by necessary feedback. labels Aug 16, 2023
@compjotr
Copy link
Author

@Witoso Screenshot 2023-08-16 at 10 29 10
This picture is better. as you see the toolbar has moved itself about 300px to high.

@Witoso
Copy link
Member

Witoso commented Aug 16, 2023

What version are you using? #5465 was fixed in the version 39.0.0 and it's possible it solves this issue.

@compjotr
Copy link
Author

nice, that solved it!

@Witoso Witoso added resolution:resolved This issue was already resolved (e.g. by another ticket). and removed pending:feedback This issue is blocked by necessary feedback. labels Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:table package:ui resolution:resolved This issue was already resolved (e.g. by another ticket). type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

2 participants