Raw Handling: unexpected results when pasting table with list from Google Docs

[danielbachhuber]

Description

I tried pasting a table from Google Docs and received unexpected results.

Google Docs

Gutenberg

Notably:

1. My merged bottom cell doesn't span full width. I assume this is blocked by Table block: Merge/Unmerge cells #15821
2. My unordered list in the bottom cell lost its formatting. To avoid going down the Table Block: Should we use InnerBlocks? #18768 rabbit hole, I'd be fine with it transforming to:

Related #32400

Step-by-step reproduction instructions

1. Copy table from https://docs.google.com/document/d/17fBRHVwvr9OUzvVR0H1S5-YJ8KKUam1kgnh4X9vtWgU/edit?usp=sharing
2. Paste table into Gutenberg
3. Observe results

Screenshots, screen recording, code snippet

No response

Environment info

• WordPress 6.2-alpha-54642-src, Gutenberg 14.5.0
• Chrome 107.0.5304.110 on Mac

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

[danielbachhuber]

To resolve this issue, I'd like to propose two separate PRs:

1. Support the colspan attribute on the underlying HTML (from Table block: Merge/Unmerge cells #15821 (comment)) with a test case for pasting from Google Docs.
2. Transform unordered and ordered lists within tables to simple list representations on paste.

[mpkelly]

To resolve this issue, I'd like to propose two separate PRs:

1. Support the colspan attribute on the underlying HTML (from Table block: Merge/Unmerge cells #15821 (comment)) with a test case for pasting from Google Docs.

I am going to start this. You can assign this issue to me if you like (I can't do that myself).

[t-hamano]

@mpkelly

Thanks for adding colspan support!

I would suggest adding further rowspan support, what do you think? I would be happy to help with the work.

[mpkelly]

@mpkelly

Thanks for adding colspan support!

I would suggest adding further rowspan support, what do you think? I would be happy to help with the work.

Sounds good to me, @t-hamano. It should be easy to use this PR as a template for rowpan too.

[t-hamano]

The PR to support rowspan attribute: #46629

[zaguiini]

@mpkelly was this fixed in #45981?

[t-hamano]

rowspan and colspan when pasting table was implemented in #45981 and #46629. However, #46775, which allows pasting of list content, has not yet been merged.

[autumnfjeld]

@mpkelly is this something that you think you'll work on soon or should we move it to the backlog? THere hasn't been any interaction in this issue in eight months.

[mpkelly]

@mpkelly is this something that you think you'll work on soon or should we move it to the backlog? THere hasn't been any interaction in this issue in eight months.

It can go on the backlog and I can take a look in the next cool down week.