Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#55006 closed defect (bug) (fixed)

Twenty Fifteen: Button border issue

Reported by: umesh84 Owned by: audrasjb
Priority: normal Milestone: 6.1
Component: Bundled Theme Version: 5.9
Severity: normal Keywords: has-patch has-screenshots commit
Cc: Focuses: ui, css

Description

Twenty Fifteen Themes Gutenberg editor button border bottom is not working.

Attachments (6)

Screenshot_4.png​ (58.7 KB ) - added by umesh84 5 years ago.
55006.patch​ (578 bytes ) - added by umesh84 5 years ago.
added patch
55006.1.patch​ (957 bytes ) - added by sabernhardt 4 years ago.
55006.2.patch​ (494 bytes ) - added by sabernhardt 4 years ago.
removing the editor stylesheet change
c29a0c00dacd46e63efc672c502fcbb1.gif​ (126.7 KB ) - added by audrasjb 4 years ago.
Before patch
86ab6d9c20d52253e59d9576e218fc72.gif​ (46.2 KB ) - added by audrasjb 4 years ago.
After patch

Download all attachments as: .zip

Change History (13)

@umesh84
5 years ago

@umesh84
5 years ago

added patch

#1 @umesh84
5 years ago

  • Keywords has-patch added; needs-patch removed

#2 @mukesh27
5 years ago

#55003 was marked as a duplicate.

#3 @SergeyBiryukov
5 years ago

  • Component General → Bundled Theme
  • Summary Twenty Fifteen Themes button border issue → Twenty Fifteen: Button border issue

@sabernhardt
4 years ago

#4 @sabernhardt
4 years ago

  • Milestone Awaiting Review → 6.1

This is related to the :where() selector, which also affected Twenty Twenty (#55824).

Additional change in editor-blocks.css:
As a follow-up to r44381, 55006.1.patch​ also corrects .is-style-outline .wp-block-button__link:not(.has-text-color), to match blocks.css.

@sabernhardt
4 years ago

removing the editor stylesheet change

#5 @sabernhardt
4 years ago

I removed the extra edit. I had noticed a problem with Gutenberg activated, but the plugin may have fixed that already with inline styles. (And if it's not fixed, we can revisit that later for other themes, too.)

#6 @audrasjb
4 years ago

  • Keywords has-screenshots commit added
  • Owner set to audrasjb
  • Status new → accepted

I tested the patch and it looks good to me (see above screenshots).

Self-assigning for commit.

#7 @audrasjb
4 years ago

  • Resolution → fixed
  • Status accepted → closed

In 54151:

Twenty Fifteen: Ensure border-bottom styles are applied to the Button Block.

This issue was related to the :where() selector, which also affected Twenty Twenty (see #55824).

Props umesh84, sabernhardt.
Fixes #55006.
See #55824.

Note: See TracTickets for help on using tickets.