Opened 4 years ago
Closed 20 months ago
#48722 closed defect (bug) (fixed)
Twenty Sixteen shows redundant borders for table in 5.3
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.5 | Priority: | normal |
Severity: | normal | Version: | 5.3 |
Component: | Bundled Theme | Keywords: | has-patch needs-testing has-screenshots |
Focuses: | css | Cc: |
Description
I don't know if Twenty Sixteen theme is still supported by WP 5.3 or if the theme problem should be reported here, but..
As of WP 5.3, the Gutenberg table block is inserted within a figure tag
<figure class="wp-block-table is-style-regular"> <table class=... ... </figure>
but only bare table is inserted in WP prior 5.3. So the CSS for displaying a figure tag will add extra borders in top & left in WP 5.3 with the Twenty Sixteen theme.
See this link for example: https://noyle.home.blog/2019/11/19/19/.
There could be other blocks start getting a figure tag wrapper around themselves which will show a top and a left border for them. And the affected themes should not be limited to Twenty Sixteen.
Attachments (12)
Change History (24)
#2
@
4 years ago
- Keywords needs-patch added
- Owner set to nielslange
- Status changed from new to assigned
#3
@
4 years ago
- Keywords has-patch needs-testing has-screenshots added; needs-patch removed
- Milestone changed from Awaiting Review to 5.3.1
#4
@
4 years ago
@noyle Thank you very much for reporting this issue! Could you test is the patch fixed the reported issue on your end?
#5
@
4 years ago
Thanks for looking into it and sorry for a separated reply.
I've applied patches for the two CSS files in my local site that is upgraded from 5.2. The block editor now shows correctly for strips style table. However in the front end, default style table now shows extra borders. Firefox calculates the border width as 3px for that <figure> block.
#6
@
4 years ago
Hi @nielslange, I've uploaded two more screenshots from a new WordPress 5.3 install with provided patch, in Firefox. Both default table and strips table have undesired borders.
HTML codos are:
<figure class="wp-block-table"><table class=""><tbody><tr><td>default</td><td>table</td></tr><tr><td>row</td><td>two</td></tr><tr><td></td><td></td></tr><tr><td>last</td><td>row</td></tr></tbody></table></figure> <figure class="wp-block-table is-style-stripes"><table class=""><tbody><tr><td>stripes</td><td>table</td></tr><tr><td>row</td><td>two</td></tr><tr><td></td><td></td></tr><tr><td>last</td><td>row</td></tr></tbody></table></figure>
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
4 years ago
#8
follow-up:
↓ 9
@
3 years ago
Hi,
@nielslange, as ticket owner and given 5.3.1 RC is scheduled for Tuesday, do you think the patch would make it for 5.3.1? Are you available to test the proposed patch? We'll need a decision on this ticket as soon as possible. Thanks!
#9
in reply to:
↑ 8
@
3 years ago
Hi,
@nielslange, as ticket owner and given 5.3.1 RC is scheduled for Tuesday, do you think the patch would make it for 5.3.1? Are you available to test the proposed patch? We'll need a decision on this ticket as soon as possible. Thanks!
Hello @audrasjb, while I cannot see a proposed patch, I can create a patch, based on @noyle's feedback. I'll do that later today.
#11
@
3 years ago
- Milestone changed from 5.4 to Future Release
Hi,
With 5.4 Beta 3 approaching and the Beta period reserved for bugs introduced during the cycle, this is being moved to Future Release. If any maintainer or committer feels this should be included or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.
#12
@
20 months ago
- Milestone changed from Future Release to 5.5
- Resolution set to fixed
- Status changed from assigned to closed
Thank you for the report and your efforts!
This apparently was fixed in r48074 (ticket:50180) for WordPress 5.5.
Editor - default style - before