Opened 5 years ago
Closed 5 years ago
#47019 closed defect (bug) (fixed)
Twenty Sixteen: Text color setting in pullquote block not applied
Reported by: | kokkieh | Owned by: | ianbelanger |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | 5.0 |
Component: | Bundled Theme | Keywords: | good-first-bug has-patch |
Focuses: | Cc: |
Description
Text color settings in the pull quote block are not being applied.
To replicate:
- With Twenty Sixteen active, create a new post in the block editor
- Add a Pullquote block
- Add some text and try adjusting the color settings
Result:
The border color is applied, but the text color is not applied in either the editor or the published post.
The source code for the post does show a class for the applied color:
<figure class="wp-block-pullquote" style="border-color:#ff675f"><blockquote class="has-text-color has-bright-blue-color"><p>This is a quote</p><cite>by me<br></cite></blockquote></figure>
But looking in the inspector it seems the color is instead being applied from the blocks.css
file, line 170:
.wp-block-pullquote p { color: #686868; font-size: 19px; font-size: 1.1875rem; }
Reported by a WordPress.com user in the forums, but confirmed on a self-hosted installation as well. Also tested with no plugins active at all to rule out possible conflicts.
Attachments (3)
Change History (22)
#1
@
5 years ago
- Keywords needs-patch good-first-bug added
- Milestone changed from Awaiting Review to 5.3
- Version changed from 5.1.1 to 5.0
#4
@
5 years ago
- Keywords 2nd-opinion added
Hello guys!
Tested both 47019 and 47019.2.
47019 looks good to me; it seems to solve the issue on both the editor and frontend. On the other hand, 47019.2 doesn't seem to work well.
First time participating/contributing so I hope I'm doing it correctly! Looking forward to help out more.
This ticket was mentioned in Slack in #core by sergey. View the logs.
5 years ago
#6
@
5 years ago
Hi all.
I also tested the 47019 patch and I confirm that it works on my freshly installed latest build.
This is my first contribution. Hope it's helpful.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
5 years ago
This ticket was mentioned in Slack in #core by marybaum. View the logs.
5 years ago
#12
@
5 years ago
- Keywords commit added; needs-testing removed
The most recent patch applies cleanly to trunk
, and the testing appears to confirm a resolution. Marking this one for commit
.
This ticket was mentioned in Slack in #core by ianbelanger. View the logs.
5 years ago
#14
@
5 years ago
- Milestone changed from 5.3 to 5.4
I might be doing something wrong, but for some reason 47019.diff did not resolve the issue in my testing. I did, however, notice that other bundled themes appear to have the same issue.
Moving to 5.4 to handle this consistently across all bundled themes.
@
5 years ago
Updates patch with with style fix for pullquote, border-left was not showing on front end
#16
@
5 years ago
- Keywords commit added; needs-testing removed
- Owner changed from SergeyBiryukov to ianbelanger
Reviewing for commit.
Thanks, @kokkieh! I am also able to reproduce this. Looks like the text color has never worked correctly in Twenty Sixeteen.