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

For specific tables, properties are not applied to some cells #17538

Open
kubaklatt opened this issue Nov 26, 2024 · 1 comment · May be fixed by #17697
Open

For specific tables, properties are not applied to some cells #17538

kubaklatt opened this issue Nov 26, 2024 · 1 comment · May be fixed by #17697
Assignees
Labels
package:table support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@kubaklatt
Copy link

📝 Provide detailed reproduction steps (if any)

  1. Open for example - https://ckeditor.com/docs/ckeditor5/latest/examples/builds-custom/full-featured-editor.html
  2. Using Source Editing apply this table:

<figure class="table" style="width:813pt;"><table class="ck-table-resized"><colgroup><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:21.16%;"><col style="width:13.65%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.52%;"><col style="width:6.51%;"></colgroup><tbody><tr><td style="height:16.0pt;width:813pt;" colspan="12">lorem</td></tr><tr><td style="border-top-style:none;height:29.0pt;width:53pt;">lorem&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;">lorem</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">lorem</td><td style="border-left-style:none;border-top-style:none;width:53pt;">lorem</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td><td style="border-top-style:none;width:53pt;">&nbsp;</td></tr><tr><td style="height:29.0pt;width:53pt;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;">lorem</td><td>&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">lorem</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-bottom:1.0pt solid black;border-right:1.0pt solid black;height:61.0pt;width:159pt;" colspan="3" rowspan="4">lorem</td><td style="border-left-style:none;">lorem</td><td>&nbsp;</td><td style="text-align:right;width:111pt;">lorem</td><td style="border-left-style:none;">&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td></tr><tr><td style="border-left-style:none;height:15.0pt;">lorem</td><td>&nbsp;</td><td style="width:111pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-left-style:none;height:15.0pt;">lorem</td><td>&nbsp;</td><td style="text-align:right;width:111pt;">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-left-style:none;height:16.0pt;">lorem</td><td>&nbsp;</td><td style="text-align:right;width:111pt;">lorem</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-right:1.0pt solid black;height:57.0pt;width:159pt;" colspan="3">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;width:225pt;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;width:164pt;" colspan="2">lorem</td><td style="border-left-style:none;width:53pt;">lorem</td><td style="border-left-style:none;width:53pt;">lorem</td><td style="border-left-style:none;width:53pt;">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;width:106pt;" colspan="2">lorem</td></tr><tr><td style="border-bottom:1.0pt solid black;height:31.0pt;width:159pt;" colspan="3" rowspan="2">lorem</td><td style="border-bottom:1.0pt solid black;border-right:1.0pt solid black;width:225pt;" colspan="2" rowspan="2">lorem</td><td style="border-bottom:1.0pt solid black;border-right:1.0pt solid black;width:164pt;" colspan="2" rowspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;width:53pt;">lorem</td><td style="border-bottom:1.0pt solid black;border-top-style:none;width:53pt;" rowspan="2">lorem</td><td style="border-bottom:1.0pt solid black;border-top-style:none;width:53pt;" rowspan="2">lorem</td><td style="border-bottom:1.0pt solid black;border-right:1.0pt solid black;width:106pt;" colspan="2" rowspan="2">lorem</td></tr><tr><td style="border-left-style:none;height:16.0pt;width:53pt;">lorem</td></tr><tr><td style="border-right:.5pt solid black;height:15.0pt;width:159pt;" colspan="3">lorem</td><td>lorem</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td><td>&nbsp;</td></tr><tr><td style="border-right:.5pt solid black;height:15.0pt;width:159pt;" colspan="3">lorem</td><td style="border-top-style:none;">lorem</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-right:.5pt solid black;height:16.0pt;width:159pt;" colspan="3">lorem</td><td style="border-top-style:none;">lorem</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td><td style="border-top-style:none;">&nbsp;</td></tr><tr><td style="border-top-style:none;height:71.0pt;width:53pt;">lorem</td><td style="border-left-style:none;width:106pt;" colspan="2">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;width:111pt;">lorem</td><td style="width:53pt;">lorem</td><td style="width:53pt;">lorem</td><td style="border-left-style:none;width:53pt;">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="3">lorem</td></tr><tr><td style="height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;">lorem</td><td style="width:53pt;">lorem</td><td style="width:53pt;">lorem</td><td style="border-left-style:none;">lorem</td><td style="border-right:.5pt solid black;" colspan="3">lorem</td></tr><tr><td style="height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="border-right:.5pt solid black;" colspan="3">lorem</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">lorem</td><td style="width:53pt;">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="border-right:.5pt solid black;" colspan="3">lorem</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:15.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="border-top-style:none;height:16.0pt;">lorem</td><td style="border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;" colspan="2">lorem</td><td style="border-left-style:none;border-top-style:none;">lorem</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="border-left-style:none;border-top-style:none;">&nbsp;</td><td style="border-right:.5pt solid black;" colspan="3">&nbsp;</td></tr><tr><td style="height:16.0pt;">lorem</td><td style="border-right:1.0pt solid black;width:106pt;" colspan="2">lorem</td><td style="border-left-style:none;border-right:1.0pt solid black;width:225pt;" colspan="2">lorem</td><td style="width:111pt;">lorem</td><td style="border-left-style:none;width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td></tr><tr><td style="border-right:1.0pt solid black;height:16.0pt;" colspan="5">lorem</td><td style="border-top-style:none;width:111pt;">lorem</td><td style="border-left-style:none;width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td><td style="width:53pt;">&nbsp;</td></tr><tr><td style="border-bottom:.5pt solid black;height:77.0pt;width:159pt;" colspan="3" rowspan="5">lorem</td><td style="width:654pt;" colspan="9">lorem</td></tr><tr><td style="height:15.0pt;width:654pt;" colspan="9">lorem</td></tr><tr><td style="border-right:.5pt solid black;height:15.0pt;" colspan="9">lorem</td></tr><tr><td style="border-right:.5pt solid black;height:15.0pt;" colspan="9">lorem</td></tr><tr><td style="border-right:.5pt solid black;height:16.0pt;" colspan="9">lorem</td></tr></tbody></table></figure>

  1. Select the entire table and set for example background in cell properties

✔️ Expected result

Background is applied to all cells

❌ Actual result

Background is not applied to some cells

Screen.Recording.2024-11-26.at.15.38.17.mov

📃 Other details

  • Browser: …
  • OS: …
  • First affected CKEditor version: …
  • Installed CKEditor plugins: …

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@kubaklatt kubaklatt added package:table type:bug This issue reports a buggy (incorrect) behavior. labels Nov 26, 2024
@aldonace-wu aldonace-wu added the support:2 An issue reported by a commercially licensed client. label Nov 26, 2024
@Mati365 Mati365 self-assigned this Dec 30, 2024
@Mati365
Copy link
Member

Mati365 commented Dec 30, 2024

Minimum reproducible table:

<table>
	<tbody>
		<tr>
			<td>lorem</td>
			<td>lorem</td>
			<td>lorem</td>
			<td>lorem</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
		<tr>
			<td colspan="2" rowspan="5">lorem</td>
			<td colspan="6">lorem</td>
		</tr>
		<tr>
			<td colspan="6">lorem</td>
		</tr>
	</tbody>
</table>

Looks like it's related to colspans, as the selection of such columns is not properly represented in the rows before

selections-tables-2024-12-30_08.12.46.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:table support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants