Make WordPress Core

Opened 21 months ago

Closed 20 months ago

Last modified 20 months ago

#56818 closed defect (bug) (fixed)

Padding within table cells ignored

Reported by: vtad's profile vtad Owned by: desrosj's profile desrosj
Milestone: 6.1.1 Priority: normal
Severity: normal Version: 6.1
Component: Editor Keywords: close
Focuses: Cc:

Description

I defined the padding (10 px) for the table-block. It shows correctly in the Editor, but it is ignored when displaying at the front end of twenty twenty three

Change History (28)

#1 @desrosj
21 months ago

  • Component changed from Themes to Bundled Theme
  • Milestone changed from Awaiting Review to 6.1
  • Summary changed from Twenty twenty three 6.1 RC1 ignores padding to Twenty Twenty-Three: 6.1 RC1 ignores padding

Thanks for this report @vtad, and welcome to Trac!

I'm moving this to the milestone to be investigated.

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


21 months ago

#3 @mikachan
20 months ago

Thanks for reporting, @vtad!

Could you please provide the block markup that you were using? You can copy this by clicking on the three-dot menu for the table block, and then clicking 'Copy block'.

I am able to see the padding I've set both in the editor and on the front end, like this:

Editor:
https://cldup.com/UlVQhocrSS.png

Front end:
https://cldup.com/m4yqNxlD_y.png

However, I'm not seeing the cell borders. It looks like these are only set within the editor, and not on the front end. I've opened an upstream issue for this: https://github.com/WordPress/gutenberg/issues/45065

This ticket was mentioned in Slack in #core by chaion07. View the logs.


20 months ago

#5 @desrosj
20 months ago

  • Keywords reporter-feedback added

#6 @mikachan
20 months ago

I think I've tracked this padding issue down to the changes in https://github.com/WordPress/gutenberg/pull/44640, where we removed the theme.css from loading in the editor if the theme does not opt into block styles. I believe this is the correct behaviour for now.

To improve this specifically for the table block, perhaps we could always include that default padding value on both the front end and in the editor since it's not currently editable via Global Styles? I've tried this in a PR here: https://github.com/WordPress/gutenberg/pull/45069

#7 @vtad
20 months ago

  • Summary changed from Twenty Twenty-Three: 6.1 RC1 ignores padding to Twenty Twenty-Three: 6.1 RC1 ignores padding within the celss

The table block

<!-- wp:table {"style":{"border":{"width":"1px"},"spacing":{"padding":{"top":"10px","right":"10px","bottom":"10px","left":"10px"}}},"borderColor":"cyan-bluish-gray","className":"is-style-stripes"} -->
<figure class="wp-block-table is-style-stripes" style="padding-top:10px;padding-right:10px;padding-bottom:10px;padding-left:10px"><table class="has-border-color has-cyan-bluish-gray-border-color" style="border-width:1px"><tbody><tr><td>1</td><td>2</td><td>3</td></tr><tr><td>DAX – Tageschart Ichimoku-Analyse</td><td>Georg </td><td>13. Okt 2022</td></tr><tr><td>DAX – ein irritierendes Interview und ein intakter Abwärtstrend</td><td>Karin Roller</td><td>15. Sep 2022</td></tr><tr><td>4</td><td>5</td><td>6</td></tr></tbody></table></figure>
<!-- /wp:table -->

Front Page : https://proto.vtad.de/

#8 @vtad
20 months ago

  • Summary changed from Twenty Twenty-Three: 6.1 RC1 ignores padding within the celss to Twenty Twenty-Three: 6.1 RC1 ignores padding within the cells

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


20 months ago

#10 @audrasjb
20 months ago

This issue was reported upstream on Gutenberg repo, but since it’s not milestoned, I don’t think it’s going to be fixed in time for 6.1.

Let’s give it one more day, then punt it to the next minor.

#11 @desrosj
20 months ago

  • Keywords reporter-feedback removed
  • Milestone changed from 6.1 to 6.1.1

There is a PR upstream attempting to improve this. While this is not ideal, it's not a blocker or serious issue for 6.1 to be released.

I'm punting to 6.1.1 so we can follow up on it in the next minor release.

#12 @JeffPaul
20 months ago

  • Keywords needs-patch added

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


20 months ago

#15 @JeffPaul
20 months ago

  • Keywords has-patch needs-testing added; needs-patch removed

#16 @nithins53
20 months ago

@mikachan can you check to see whether the Add citation text is aligned at the center or does it move to the left?

#17 @mikachan
20 months ago

Hi @nithins53, do you mean the caption text in the Table block? It looks like this is centered for me in the editor, and it is displayed on the left on the front end:

Editor Front end
https://cldup.com/rWuzqXxFHj.png https://cldup.com/2qQb3lxuxP.png

It's also a different font style. These should really be the same between the editor and front end. I'm happy to take a look at this, unless you'd like to?

#18 @nithins53
20 months ago

@mikachan Yes, I was referring to the Add Caption text. Upon testing, I found it to be aligned to the left. It would be nice if you could take a look into it.

#19 @desrosj
20 months ago

  • Keywords close added

The noted upstream PR has been merged on GitHub and marked for merging into SVN.

Marking as close since this will be resolved when that happens.

#20 @peterwilsoncc
20 months ago

  • Keywords has-patch needs-testing removed

Keyword tidyup for the workflow report.

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


20 months ago

#22 @JeffPaul
20 months ago

  • Component changed from Bundled Theme to Editor

#23 @desrosj
20 months ago

  • Summary changed from Twenty Twenty-Three: 6.1 RC1 ignores padding within the cells to Padding within table cells ignored

#24 @desrosj
20 months ago

  • Owner set to desrosj
  • Resolution set to fixed
  • Status changed from new to closed

In 54811:

Editor: Update block editor packages to the latest patch releases.

This updates the block editor related npm dependencies to their latest patch versions ahead of WordPress 6.1.1.

For a full list of what’s included in this update, see https://github.com/WordPress/gutenberg/compare/432ed388f8d0614f9de775738b24b0ea96664715...6566f5fe9ece6ad5ae550349d3b1f0944a011040.

Props aaronrobertshaw, ntsekouras, bernhard-reiter, ramonopoly, isabel_brison, andrewserong, get_dave, scruffian, andraganescu, talldanwp, mciampini, noisysocks, cbravobernal, bph, tyxla, ellatrix, czapla, mcsf, ironprogrammer, wildworks, peterwilsoncc, mamaduka, mikachan, spacedmonkey, cybr, youknowriad, alexstine, aristath, kevin940726, ndiego, 0mirka00, poena, joen, ryankienstra, desrosj, vtad, nithins53, audrasjb, kacper3355, sabernhardt.
Fixes #57038, #56818, #56955, #56923.

#25 @desrosj
20 months ago

In 54812:

Editor: Update block editor packages to the latest patch releases.

This updates the block editor related npm dependencies to their latest patch versions ahead of WordPress 6.1.1.

For a full list of what’s included in this update, see https://github.com/WordPress/gutenberg/compare/432ed388f8d0614f9de775738b24b0ea96664715...6566f5fe9ece6ad5ae550349d3b1f0944a011040.

Props aaronrobertshaw, ntsekouras, bernhard-reiter, ramonopoly, isabel_brison, andrewserong, get_dave, scruffian, andraganescu, talldanwp, mciampini, noisysocks, cbravobernal, bph, tyxla, ellatrix, czapla, mcsf, ironprogrammer, wildworks, peterwilsoncc, mamaduka, mikachan, spacedmonkey, cybr, youknowriad, alexstine, aristath, kevin940726, ndiego, 0mirka00, poena, joen, ryankienstra, desrosj, vtad, nithins53, audrasjb, kacper3355, sabernhardt.
Merges [54811] to the 6.1 branch.
Fixes #57038, #56818, #56955, #56923.

#26 @desrosj
20 months ago

#56972 was marked as a duplicate.

#27 @desrosj
20 months ago

#56959 was marked as a duplicate.

#28 @desrosj
20 months ago

#56995 was marked as a duplicate.

Note: See TracTickets for help on using tickets.