#63520 closed defect (bug) (fixed)
Twenty Fourteen: Latest Comments block link color does not belong in Content Sidebar
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.8.2 | Priority: | normal |
| Severity: | normal | Version: | 5.8 |
| Component: | Bundled Theme | Keywords: | has-patch has-test-info |
| Focuses: | css | Cc: |
Description
[51205] set the link color to white for the Latest Comments block within any widget area. That was good for the Primary Sidebar and Footer Widget Area with their dark backgrounds, but it does not fit the Content Sidebar.
Attachments (4)
Change History (20)
This ticket was mentioned in PR #8888 on WordPress/wordpress-develop by @sabernhardt.
8 months ago
#1
- Keywords has-patch added
@sabernhardt commented on PR #8888:
8 months ago
#2
#3
@
8 months ago
Test Report
Description
This report validates that the indicated patch works as expected.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/8888.diff
Environment
- WordPress: 6.9-alpha-60093-src
- PHP: 8.2.22
- Server: nginx/1.27.0
- Database: mysqli (Server: 8.0.39 / Client: mysqlnd 8.2.22)
- Browser: Chrome 136.0.0.0
- OS: Linux
- Theme: Twenty Fourteen 4.2
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
- ✅ Issue resolved with patch.
Supplemental Artifacts
Added as Attachment
#4
@
8 months ago
Reproduction Report
Description
This report validates whether the issue can be reproduced.
Environment
- WordPress: 6.8.1
- PHP: 8.2.23
- Server: nginx/1.26.1
- Database: mysqli (Server: 8.0.35 / Client: mysqlnd 8.2.23)
- Browser: Chrome 137.0.0.0
- OS: Windows 10/11
- Theme: Twenty Fourteen 4.2
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
- ✅ Error condition occurs (reproduced).
Additional Notes
Steps to reproduce the issue:
- Add 'recent comment widget' in Primary Sidebar, Footer Widget Sidebar & Content Sidebar from Appearance > Widgets
- Now visit the front blog listing and blog detail page. You can see that 'Comment Author' name text and Post Title text are now showing in Content Sidebar because it have white background color.
- Those are showing properly in Primary Sidebar and Footer Widget Sidebar.
Supplemental Artifacts
Screenshot: https://prnt.sc/6sIJaaA1frC-
#5
@
8 months ago
- Keywords has-test-info added
Test Report
Description
This report validates whether the indicated patch works as expected.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/8888.diff
Environment
- WordPress: 6.8.1
- PHP: 8.2.23
- Server: nginx/1.26.1
- Database: mysqli (Server: 8.0.35 / Client: mysqlnd 8.2.23)
- Browser: Chrome 137.0.0.0
- OS: Windows 10/11
- Theme: Twenty Fourteen 4.2
- MU Plugins: None activated
- Plugins:
- Test Reports 1.2.0
Actual Results
- ✅ Issue resolved with patch.
Additional Notes
- This css patch is working fine.
Supplemental Artifacts
Screenshot: https://prnt.sc/uhu0QTAGmpna
#6
@
8 months ago
- Milestone changed from Awaiting Review to 6.9
I think assigning the color only to the Primary and Footer widget areas would have been a better choice four years ago, when blocks were introduced to widgets.
.primary-sidebar .wp-block-latest-comments__comment-meta a,
.footer-sidebar .wp-block-latest-comments__comment-meta a {
color: #fff;
}
However, a site with a child theme can register an extra widget area. For that situation, PR 8888 expects that any custom sidebar is against a dark background, does not include a Latest Comments widget, and/or already has custom styles to fix the text color. The patch only excludes the Content Sidebar:
.widget-area:where(:not(.content-sidebar)) .wp-block-latest-comments__comment-meta a {
color: #fff;
}
#7
@
8 months ago
Test Report
This report validates that the indicated patch addresses the issue.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/8888.diff
Environment
- OS: Windows 11 Home
- Web Server: Nginx 1.27.5
- PHP: 8.2.28
- WordPress: 6.9-alpha-60093-src
- Browser: Mozilla Firefox 139.0.1 (Latest)
- Theme: Twenty Fourteen
- Active Plugins:
- Code Snippets 3.6.8
- WordPress Beta Tester 3.6.3
Actual Results
- ✅ Issue resolved with patch – color in content sidebar is green as mentioned.
Additional Notes
- Kindly refer attached images for before and after screenshots.
@sandeepdahiya commented on PR #8888:
8 months ago
#8
Test Report
This report validates that the indicated patch addresses the issue.
Patch tested: https://patch-diff.githubusercontent.com/raw/WordPress/wordpress-develop/pull/8888.diff
Environment
- OS: Windows 11 Home
- Web Server: Nginx 1.27.5
- PHP: 8.2.28
- WordPress: 6.9-alpha-60093-src
- Browser: Mozilla Firefox 139.0.1 (Latest)
- Theme: Twenty Fourteen
- Active Plugins:
- - Code Snippets 3.6.8
- - WordPress Beta Tester 3.6.3
Actual Results
- ✅ Issue resolved with patch – color in content sidebar is green as mentioned.
Additional Notes
- Kindly refer attached images for before patch and after patch results.
Before applying patch: -
After applying patch: -
#9
@
7 months ago
- Keywords commit added
- Owner set to karmatosed
- Status changed from new to assigned
Assigning to myself to follow through and see about commit.
#12
@
7 months ago
- Keywords commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for backport consideration.
This ticket was mentioned in Slack in #core-themes by sabernhardt. View the logs.
7 months ago
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
7 months ago
@sabernhardt commented on PR #8888:
6 months ago
#16
Committed in r60368
Avoids adding white link text where the background color is white.
Trac 63520