Make WordPress Core

Opened 22 months ago

Closed 4 months ago

#56601 closed defect (bug) (worksforme)

Twenty Ten: overlap issue in Button block

Reported by: kajalgohel's profile kajalgohel Owned by:
Milestone: Priority: normal
Severity: normal Version: 6.0.2
Component: Bundled Theme Keywords: has-patch needs-testing 2nd-opinion close
Focuses: ui, css Cc:

Description

In Twenty Ten Theme, when we add Button block in editor side and change alignment as "Align left" or "Align right". After that we add paragraph block or any other block then we can see that the Button block is overlapping with the newly added block.

Steps to replicate:
1: Activate the Twenty Ten Theme
2: Add Button block
3: Choose "Align right" or "Align left" from Align option
4: Add paragraph Block
5: View the page/post at editor side

For better understanding I provide video attachment link.
Video link: https://share.cleanshot.com/XuFtIRCURbgfcSSNxf41

Thanks

Attachments (1)

#56601.patch (354 bytes) - added by kajalgohel 22 months ago.
patch added

Download all attachments as: .zip

Change History (6)

@kajalgohel
22 months ago

patch added

#1 @kajalgohel
22 months ago

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

This ticket was mentioned in Slack in #core-test by monksbistro. View the logs.


21 months ago

#3 @pooja1210
21 months ago

Hi @kajalgohel,

Was able to reproduce issue and have cross checked the patch as well and following is the test report.

Test Report

Env

  • WordPress 6.0.2
  • Browser Chrome
  • Mac Big Sur
  • Theme: Twenty Ten
  • Gutenberg Editor

The patch is working as expected.

Version 0, edited 21 months ago by pooja1210 (next)

#4 @sabernhardt
14 months ago

  • Keywords 2nd-opinion close added

The block toolbar's alignment button uses text alignment classes instead of alignleft or alignright now (see GB45663). I think we can consider this fixed in the editor, without a theme patch.

#5 @karmatosed
4 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to worksforme
  • Status changed from new to closed

I agree with your opinion here @sabernhardt that closing is ideal as this is fixed in the editor without a theme patch. Thank you everyone for collaborating on this.

Note: See TracTickets for help on using tickets.