1664350 - Exporting `<table>` including `<svg>` to the clipboard is broken for "text/html"

Status: NEW

(Core :: DOM: Serializers, defect)

Description

[gonghan.zou]

Attachment: table-example.html

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.102 Safari/537.36

Steps to reproduce:

Firefox does not copy html structure of tables correctly, when there are SVG images in table header.

Open table-example.html in attachment in firefox,
Select from "A" to "B"
Inspect "text/html" data in your clipboard.

Actual results:

The "text/html" data in your clipboard does not match the original table structure:
The contents of table header are outside the table, but it should be inside table.

Expected results:

It should preserve the correct html table structure.

Comment 1

[gonghan.zou]

Just some updates,

• Seems replace the "SVG" with an "IMG" does not fix this issue.
• "Control + A" will copy two tables instead of one.

Our current workaround is a DIV with background image.

Comment 2

[Mirko Brodesser (:mbrodesser-Igalia)]

Reproducible with Nightly on Ubuntu 18.04. Given the old existing bugs in <table>-Selection code, I suspect the bug exists since a longer time (years, presumably). Works with Chrome.

Comment 3

[Mirko Brodesser (:mbrodesser-Igalia)]

This issue isn't <th>-specific, it also happens when changing them to <td>s.

Selecting the whole table by taping Ctrl+a and copying it to the clipboard works with Chrome.

Comment 4

[Mirko Brodesser (:mbrodesser-Igalia)]

Couldn't reproduce the issue when replacing the <svg> with an <img>.

:gonghan.zou: is it still reproducible for you, when using an <img>? Moreover, it'd be helpful to know the use-cases for copying <table>s including <svg>s.

Comment 5

[Mirko Brodesser (:mbrodesser-Igalia)]

(In reply to Mirko Brodesser (:mbrodesser) from comment #4)

Couldn't reproduce the issue when replacing the <svg> with an <img>.

:gonghan.zou: is it still reproducible for you, when using an <img>?

I suspect the same issue as in bug 1669050 is meant.