Make WordPress Core

Opened 2 years ago

Closed 7 weeks ago

#56157 closed defect (bug) (fixed)

Twenty Twenty: Table font size not reflected in frontend

Reported by: umesh84's profile umesh84 Owned by: karmatosed's profile karmatosed
Milestone: 6.7 Priority: normal
Severity: normal Version: 6.0
Component: Bundled Theme Keywords: has-screenshots has-patch needs-testing commit
Focuses: css Cc:

Description

Twenty Twenty Table font size not reflected in front.

Steps to replicate:
1: Activate the Twenty Twenty Themes

2: add Table block

3: Select Size

URL: https://www.loom.com/share/dc28c0c09ef247e1afe55a0561c1c911

Attachments (4)

#56157.patch (396 bytes) - added by umesh84 2 years ago.
added patch
56157.2.patch (1014 bytes) - added by sabernhardt 15 months ago.
inheriting font size only when user selects a size in the Table block
2020-Table-block-font-size-front.png (112.8 KB) - added by sabernhardt 7 weeks ago.
(without patch) custom font sizes still do not override the theme's table styles on the front end
SCR-20240729-kbru.png (51.1 KB) - added by karmatosed 7 weeks ago.
Showing front end

Download all attachments as: .zip

Change History (22)

@umesh84
2 years ago

added patch

This ticket was mentioned in PR #2953 on WordPress/wordpress-develop by pratiweb.


2 years ago
#1

  • Keywords has-patch added; needs-patch removed

Solved issue #56157

Change font size to inherit in Twenty Twenty Table style.

#2 @SergeyBiryukov
2 years ago

  • Summary changed from Themes: Twenty Twenty Table font size not reflected in frontend to Twenty Twenty: Table font size not reflected in frontend

@sabernhardt
15 months ago

inheriting font size only when user selects a size in the Table block

#3 @sabernhardt
15 months ago

#58489 was marked as a duplicate.

#4 @sabernhardt
15 months ago

  • Keywords needs-testing added

The font size needs to stay the same for any tables or Table blocks that do not use the custom block setting.

#5 @shailu25
15 months ago

  • Keywords has-screenshots added

Test Report For https://core.trac.wordpress.org/attachment/ticket/56157/56157.2.patch Patch.

Table blocks that do not use the custom block setting:
=============================================

Back-end : https://prnt.sc/SaHZm1N4S9R4
Front-end : https://prnt.sc/XaLdVqdWIJPc

Table Block when user selects a font size:
===================================

Backend: https://prnt.sc/tDHPsfHRZJ_d

Frontend :
=========
Before Patch Frontend: https://prnt.sc/mLm3vEz9V3Mp
After Patch Frontend : https://prnt.sc/IH4Vk28EVTTg

https://core.trac.wordpress.org/attachment/ticket/56157/56157.2.patch Patch is Working Fine.

Result After Patch:

  • Table font size is reflected in frontend same as backend (When User select a custom font size)
  • Table blocks font size stay same when do not use the custom block setting

#6 @karmatosed
2 months ago

Assigning so I can review this and look to see if it still is an issue, that the patch works and if it could be a candidate for commit.

#7 @karmatosed
2 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 6.7

#8 @karmatosed
2 months ago

  • Keywords close added; needs-testing commit removed

In testing this, the issue has been resolved with previous patches or at least seems to have been. I tested it both using 10 and 100px in custom size and both times it was without the patch working.

I am going to recommend this is closed for now but very happy for others to correct that if they can replicate the issue. To be clear I followed these instructions:

1: Activate the Twenty Twenty Themes

2: add Table block

3: Select Size

#9 @karmatosed
2 months ago

  • Milestone changed from 6.7 to Future Release

#10 @karmatosed
8 weeks ago

  • Milestone Future Release deleted
  • Resolution set to worksforme
  • Status changed from new to closed

Thank you everyone. For now I am going to close this.

@sabernhardt
7 weeks ago

(without patch) custom font sizes still do not override the theme's table styles on the front end

#11 @sabernhardt
7 weeks ago

  • Keywords needs-testing added; close removed
  • Milestone set to Future Release
  • Resolution worksforme deleted
  • Status changed from closed to reopened

Changing the font size already works in the editor, but still not on the front.

#12 @karmatosed
7 weeks ago

  • Owner set to karmatosed
  • Resolution set to fixed
  • Status changed from reopened to closed

In 58819:

Twenty Twenty: Calendar and Table blocks do not apply custom font size.

This fixes adding a custom font size to a Calendar and Table block. This was only an issue for custom font size entering.

Props nidhidhandhukiya, yurajsinj2211, ankit-k-gupta, anveshika, sabernhardt, darshitrajyaguru97, shailu25, umesh84, SergeyBiryukov.
Fixes #59996, #56157.

#13 @shailu25
7 weeks ago

@karmatosed

56157.2.patch changes are pending in this ticket.

Because in Changeset 58819 only Calander block changes are added Still table block font size changes are in pending.

Last edited 7 weeks ago by shailu25 (previous) (diff)

#14 @karmatosed
7 weeks ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

@shailu25 very happy to reopen but I did see it working for custom font sizes. Let's get some testing on the reopen with the commit now in.

#15 @shailu25
7 weeks ago

Steps to Replicate the Table Block Font Size Issue in the Frontend:

  1. Activate the Twenty Twenty theme.
  2. Add a Table block in a page or post.
  3. Select a custom font size (e.g., 32 or Large Font) and save the page or post.
  4. Check the frontend; you will see that the custom font size is not applied to the Table block.

#16 @karmatosed
7 weeks ago

Ok going back into testing for this with those steps, thank you.

#17 @karmatosed
7 weeks ago

  • Keywords commit added
  • Milestone changed from Future Release to 6.7
  • Status changed from reopened to assigned

I can confirm in testing the latest patch works and that the front end and back end now have the custom font changes. I am therefore going to move this to commit. Thank you everyone and thank you for catching it wasn't cleared up in the previous solutions.

@karmatosed
7 weeks ago

Showing front end

#18 @karmatosed
7 weeks ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 58824:

Twenty Twenty: Fixes Table font size when custom showing on front.

The Table block was not reflecting the custom font size on the front. This solution now brings custom font sizes in for front the same as back in the editor.

Props umesh84, SergeyBiryukov, sabernhardt, shailu25.
Fixes #56157.

Note: See TracTickets for help on using tickets.