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

Merging cells in header do not update heading rows properly #6667

Closed
jodator opened this issue Apr 24, 2020 · 1 comment Β· Fixed by #6764
Closed

Merging cells in header do not update heading rows properly #6667

jodator opened this issue Apr 24, 2020 · 1 comment Β· Fixed by #6764
Assignees
Labels
package:table type:bug This issue reports a buggy (incorrect) behavior.

Comments

@jodator
Copy link
Contributor

jodator commented Apr 24, 2020

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

  1. Open "coffee table" sample.
  2. Merge cells "milk" and two below.

βœ”οΈ Expected result

The table features works properly (merging cells in body, setting heading rows, etc).

❌ Actual result

Table has one heading row (visually) but model have heading rows = 2.

πŸ“ƒ Other details

I don't know which command should take care of this (`TableUtils.removeRow()`, commands or a post-fixer).

Similar thing happen on v18.0.0 using merge table cell left/right/up/down buttons:


If you'd like to see this fixed sooner, add a πŸ‘ reaction to this post.

@jodator jodator added type:bug This issue reports a buggy (incorrect) behavior. package:table labels Apr 24, 2020
@jodator
Copy link
Contributor Author

jodator commented Apr 24, 2020

Relates to: #6630.

@Mgsy Mgsy added this to the backlog milestone Apr 27, 2020
@niegowski niegowski modified the milestones: backlog, iteration 32 May 7, 2020
@niegowski niegowski self-assigned this May 7, 2020
jodator added a commit that referenced this issue May 13, 2020
Fix (table): Table heading rows should be properly updated after removing rows as a side effect of merging cells. Closes #6667.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:table type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
3 participants