Opened 7 years ago
Closed 7 years ago
#47019 closed defect (bug) (fixed)
Twenty Sixteen: Text color setting in pullquote block not applied
| Reported by: | kokkieh | Owned by: | ianbelanger |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.4 |
| Component: | Bundled Theme | Version: | 5.0 |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: |
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
@
7 years ago
- Keywords needs-patch good-first-bug added
- Milestone Awaiting Review → 5.3
- Version 5.1.1 → 5.0
#4
@
7 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.
7 years ago
#6
@
7 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.
7 years ago
This ticket was mentioned in Slack in #core by marybaum. View the logs.
7 years ago
#12
@
7 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.
7 years ago
#14
@
7 years ago
- Milestone 5.3 → 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.
@
7 years ago
Updates patch with with style fix for pullquote, border-left was not showing on front end
#16
@
7 years ago
- Keywords commit added; needs-testing removed
- Owner changed from to
Reviewing for commit.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks, @kokkieh! I am also able to reproduce this. Looks like the text color has never worked correctly in Twenty Sixeteen.