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

Adding column next to down+left merged cell works incorrectly #5882

Closed
scofalik opened this issue Dec 2, 2019 · 2 comments · Fixed by ckeditor/ckeditor5-table#289
Closed

Adding column next to down+left merged cell works incorrectly #5882

scofalik opened this issue Dec 2, 2019 · 2 comments · Fixed by ckeditor/ckeditor5-table#289
Assignees
@niegowski
Labels
intro Good first ticket. package:table type:bug This issue reports a buggy (incorrect) behavior.
Milestone
iteration 31 (v19...

Comments

@scofalik
Copy link
Contributor

scofalik commented Dec 2, 2019 •
edited by Mgsy
Loading

📝 Provide detailed reproduction steps (if any)

  1. Create a cell that is merged down and merged left (see GIF).
  2. Insert column directly on the left of that cell.

Dec-02-2019 15-25-18

❌ Actual result

The table layout goes crazy.

If you'd like to see this fixed sooner, add a 👍 reaction to this post.
@scofalik scofalik added type:bug This issue reports a buggy (incorrect) behavior. package:table labels Dec 2, 2019
@Mgsy Mgsy added this to the backlog milestone Dec 3, 2019
@FilipTokarski FilipTokarski mentioned this issue Mar 4, 2020
Closed
@Reinmar
Copy link
Member

Reinmar commented Mar 26, 2020

Merged #6376 into this issue.

@Reinmar Reinmar added the intro Good first ticket. label Mar 26, 2020
@Reinmar Reinmar modified the milestones: backlog, iteration 31 Mar 26, 2020
@Reinmar
Copy link
Member

Reinmar commented Mar 26, 2020

Let's give this max 1 day and see if we can fix it.

@niegowski niegowski mentioned this issue Mar 27, 2020
Merged
jodator added a commit to ckeditor/ckeditor5-table that referenced this issue Mar 30, 2020
@jodator
Merge pull request #289 from ckeditor/i/5882
d239f69 
Fix: Inserting table column at the first col-row spanned cell should work properly. Closes ckeditor/ckeditor5#5882.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niegowski niegowski

Labels
intro Good first ticket. package:table type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Milestone
iteration 31 (v19.0.0)
5 participants
@Reinmar @niegowski @scofalik @mlewand @Mgsy