#55167 closed defect (bug) (fixed)
Twenty Sixteen: Fix button styles outside of entry-content
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.0 | Priority: | normal |
Severity: | minor | Version: | 5.0 |
Component: | Bundled Theme | Keywords: | has-patch has-screenshots commit |
Focuses: | css | Cc: |
Description
The buttons in the sidebar on the Twenty Sixteen theme do not correctly apply custom colors. It looks like this is because the custom color CSS selectors are specific to .entry-content.
This was originally part of an issue in the themes repo: https://github.com/Automattic/themes/issues/4580
Attachments (3)
Change History (9)
This ticket was mentioned in PR #2317 on WordPress/wordpress-develop by mikachan.
3 years ago
#1
#2
@
3 years ago
- Milestone changed from Awaiting Review to 6.0
- Version set to 5.0
Test Report
Env
- Web Server: Apache
- WordPress: 6.0-alpha-52448-src
- Browser: Chrome
- OS: Linux
- Theme: Twenty Sixteen
- Plugins: None activated
Steps to test
- Navigate to
Appearance > Themes
. - Activate
Twenty Sixteen
. - Navigate to
Appearance > Widgets
. - Add a
Buttons
block toSidebar
. - Enter a label for the button, set the
Color
toYellow
and theBackground Color
toBright Red
. - Click
Update
. - Navigate to the frontend. The button will appear with white text and a grey background.
- Apply PR 2317.
- Do a hard refresh the frontend. The button should appear with yellow text and a bright red background.
Results
- Before PR 2317, the button does not use the selected colours. ✅
- After PR 2317 the button uses the selected colours. ✅
Notes
- Introduced in 44306.
#3
@
3 years ago
- Owner set to audrasjb
- Status changed from new to reviewing
Self assigning for review.
#4
@
3 years ago
- Keywords has-screenshots commit added
- Status changed from reviewing to accepted
The patch looks good to me. Let's ship this change 🚀
3 years ago
#6
Committed in https://core.trac.wordpress.org/changeset/52926
Note: See
TracTickets for help on using
tickets.
Trac ticket: https://core.trac.wordpress.org/ticket/55167
This applies the default button styles to all button blocks in Twenty Sixteen, instead of only targeting those in entry-content. This means that any buttons added outside of .entry-content, such as in a widget area, will be styled the same as any other button blocks. This change also makes the theme custom colors work outside of .entry-content.
To test, add a button to a widget area. Make sure this button is styled the same as a button in a page or post, and that custom styles can still be applied correctly.
This fix was originally part of an issue in the themes repo: https://github.com/Automattic/themes/issues/4580