Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#54317 closed defect (bug) (fixed)

Twenty Sixteen: Table block does not respect the alignment options

Reported by: robertghetau's profile robertghetau Owned by: desrosj's profile desrosj
Milestone: 5.9 Priority: normal
Severity: normal Version: 5.0
Component: Bundled Theme Keywords: has-patch commit
Focuses: css Cc:

Description

I can see the alignment options(left/center/right) options are not being respected and they all default to left.

Attachments (3)

Markup 2021-10-25 at 12.55.57.png (244.9 KB) - added by robertghetau 2 years ago.
54317.patch (618 bytes) - added by umesh84 2 years ago.
I have fixed bug Twenty Sixteen: Table block does not respect the alignment options
54317.1.patch (703 bytes) - added by sabernhardt 2 years ago.
removing text alignment from .wp-block-table styles

Download all attachments as: .zip

Change History (10)

#2 @robertghetau
2 years ago

Sure! I've added an editor vs preview/published view.

#3 @cezarygm
2 years ago

please set something for this:

    text-align: center;
    display: flex;
    justify-content: center;
    flex-wrap: wrap;
}

@umesh84
2 years ago

I have fixed bug Twenty Sixteen: Table block does not respect the alignment options

#4 @umesh84
2 years ago

  • Keywords has-patch added

@sabernhardt
2 years ago

removing text alignment from .wp-block-table styles

#5 @sabernhardt
2 years ago

  • Component changed from General to Bundled Theme
  • Focuses css added
  • Milestone changed from Awaiting Review to 5.9
  • Version changed from 5.8.1 to 5.0

Thanks for the report and the patch!

I would prefer removing the special styles in block.css that currently override the alignment classes.

This still applies when a cell has no alignment specified:

caption,
th,
td {
	font-weight: normal;
	text-align: left;
}

(and in RTL languages, cell text is aligned right by default)

#6 @sabernhardt
2 years ago

  • Keywords commit added

#7 @desrosj
2 years ago

  • Owner set to desrosj
  • Status changed from new to reviewing

#8 @desrosj
2 years ago

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

Fixed in [52210]. Thanks everyone!

Note: See TracTickets for help on using tickets.