Make WordPress Core

Opened 3 years ago

Closed 8 months ago

#56565 closed defect (bug) (fixed)

Twenty Sixteen: Quote Block Border Left color not reflected in editor

Reported by: umesh84's profile umesh84 Owned by: karmatosed's profile karmatosed
Milestone: 6.7 Priority: normal
Severity: minor Version: 6.0.2
Component: Bundled Theme Keywords: has-patch has-screenshots commit
Focuses: ui, css Cc:

Description

Theme Twenty Sixteen Quote Block Border Left color not reflected in editor
Steps to replicate:

1: Activate the Twenty Sixteen Themes
2: add Quote block
3: Select color

For better understanding I provide video attachment link.
Video link: https://www.loom.com/share/bc19aee1e3cb4370a0642a403c61942f

Attachments (2)

56565.patch (501 bytes) - added by umesh84 3 years ago.
added patch
56565.1.patch (577 bytes) - added by sabernhardt 21 months ago.
using the .wp-block-quote class and moving style among other Quote block styles

Download all attachments as: .zip

Change History (11)

@umesh84
3 years ago

added patch

#1 @umesh84
3 years ago

  • Keywords has-patch added; needs-patch removed

#2 @desrosj
2 years ago

  • Summary changed from Theme Twenty Sixteen Quote Block Border Left color not reflected in editor to Twenty Sixteen: Quote Block Border Left color not reflected in editor

@sabernhardt
21 months ago

using the .wp-block-quote class and moving style among other Quote block styles

#3 @sabernhardt
21 months ago

  • Severity changed from normal to minor

The original theme design did not have the same color for the border as the quote text, but it has been that way in the Quote block for about two years.

With WordPress 5.0, Twenty Sixteen had a black border color and #1a1a1a dark gray citation color while the quote text was in the #686868 medium gray. By 5.3, the theme.css gave its own dark gray border (GB13248), and later it was revised to match the text color in GB26684.

In case we want to match the current block styling, I updated the selector in the patch.

#4 @shailu25
21 months ago

  • Keywords has-screenshots added

Test Report For https://core.trac.wordpress.org/attachment/ticket/56565/56565.1.patch

Before Patch: https://prnt.sc/n7UigrkW1xZe

After Patch : https://prnt.sc/D-_YULQcYOQq

Result After Patch:

  • Quote Block Border Left color is reflected in editor.

https://core.trac.wordpress.org/attachment/ticket/56565/56565.1.patch is working fine.

#5 @sabernhardt
15 months ago

#60166 was marked as a duplicate.

#6 @sabernhardt
14 months ago

#60273 was marked as a duplicate.

#7 @karmatosed
8 months ago

  • Owner set to karmatosed
  • Status changed from new to assigned

I am going to assign this to myself to test and see if this does impact design or would work to commit.

#8 @karmatosed
8 months ago

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

I think it is fair to expect this to change based on color particularly with the way that the editor works now. I am therefore going to add in the patch that is up. What this does is reflect the color change in border also.

#9 @karmatosed
8 months ago

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

In 58728:

Twenty Sixteen: Fixes quote block border not changing color.

The border was not changing color when it was changed. This brings that in. It is worth noting the original theme design did not have this but as noted in the ticket this has been in the quote block for about two years so matching the current block styling in this case makes sense.

Props umesh84, desrosj, sabernhardt.
Fixes #56565.

Note: See TracTickets for help on using tickets.