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.

Sign up for GitHub

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

Jump to bottom

Table balloon toolbar hides the first row of table #14667

Closed
dufipl opened this issue Jul 26, 2023 · 0 comments · Fixed by #14755
Closed

Table balloon toolbar hides the first row of table #14667

dufipl opened this issue Jul 26, 2023 · 0 comments · Fixed by #14755
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:table squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Milestone
iteration 67 (v40...

Comments

@dufipl
Copy link
Contributor

dufipl commented Jul 26, 2023

📝 Provide detailed reproduction steps (if any)

  1. Open http://localhost:8125/ckeditor5/tests/manual/all-features.html
  2. Insert table

✔️ Expected result

The table toolbar does not cover table rows:
image

❌ Actual result

The table toolbar does cover the first row of the table, which makes the user unable to type there:
image

📃 Other details

  • Browser: Any
    Regression on master compared to 38.1.1. As it caused E2E tests to fail yesterday, it is safe to assume that it is caused by one of the changes merged into master around 25th July 2023.

If you'd like to see this fixed sooner, add a 👍 reaction to this post.
@dufipl dufipl added type:bug This issue reports a buggy (incorrect) behavior. package:table domain:ui/ux This issue reports a problem related to UI or UX. type:regression This issue reports a bug that was not present in the previous releases. squad:features Issue to be handled by the Features team. labels Jul 26, 2023
@mlewand mlewand mentioned this issue Jul 26, 2023
Closed
@pszczesniak pszczesniak mentioned this issue Aug 25, 2023
Merged
@CKEditorBot CKEditorBot added this to the iteration 67 milestone Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
domain:ui/ux This issue reports a problem related to UI or UX. package:table squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Projects
None yet
Milestone
iteration 67 (v40.0.0)
Development

Successfully merging a pull request may close this issue.

Improvements to getOptimalPosition to handle cases with scrollable ancestors ckeditor/ckeditor5
2 participants
@CKEditorBot @dufipl