Make WordPress Core

Opened 7 weeks ago

Closed 12 days ago

#63520 closed defect (bug) (fixed)

Twenty Fourteen: Latest Comments block link color does not belong in Content Sidebar

Reported by: sabernhardt's profile sabernhardt Owned by: karmatosed's profile karmatosed
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)

before.png (151.0 KB) - added by dhruvang21 7 weeks ago.
Before patch
after.png (158.2 KB) - added by dhruvang21 7 weeks ago.
before patch.png (65.7 KB) - added by sandeepdahiya 6 weeks ago.
Before applying the patch
after patch.png (70.0 KB) - added by sandeepdahiya 6 weeks ago.
After applying the patch

Download all attachments as: .zip

Change History (19)

This ticket was mentioned in PR #8888 on WordPress/wordpress-develop by @sabernhardt.


7 weeks ago
#1

  • Keywords has-patch added

Avoids adding white link text where the background color is white.

Trac 63520

@sabernhardt commented on PR #8888:


7 weeks ago
#2

Before patch: the text color for Latest Comments links matches the Content Sidebar background (without :hover or :focus).

https://github.com/user-attachments/assets/e4b1fa12-36a4-49d7-b7ed-4b6fbca57add

With patch: the links are green in the Content Sidebar, but they remain white against a dark background in the Primary and Footer sidebars.

https://github.com/user-attachments/assets/30d5bb0d-2b7b-409c-9f5c-3e908826f004

#3 @dhruvang21
7 weeks 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

  1. ✅ Issue resolved with patch.

Supplemental Artifacts

Added as Attachment

@dhruvang21
7 weeks ago

Before patch

@dhruvang21
7 weeks ago

#4 @ravigadhiyawp
7 weeks 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

  1. ✅ 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-

Last edited 7 weeks ago by ravigadhiyawp (previous) (diff)

#5 @ravigadhiyawp
7 weeks 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

  1. ✅ Issue resolved with patch.

Additional Notes

  • This css patch is working fine.

Supplemental Artifacts

Screenshot: https://prnt.sc/uhu0QTAGmpna

#6 @sabernhardt
7 weeks 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;
}

@sandeepdahiya
6 weeks ago

Before applying the patch

@sandeepdahiya
6 weeks ago

After applying the patch

#7 @sandeepdahiya
6 weeks 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:


6 weeks 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: -

https://github.com/user-attachments/assets/956cb477-b4a9-4b4b-b51f-c1a172e8fee2

After applying patch: -

https://github.com/user-attachments/assets/92734c30-b86d-402e-87a2-1973c37f30bf

#9 @karmatosed
3 weeks ago

  • Keywords commit added
  • Owner set to karmatosed
  • Status changed from new to assigned

Assigning to myself to follow through and see about commit.

#10 @karmatosed
3 weeks ago

  • Milestone changed from 6.9 to 6.8.2

#11 @karmatosed
3 weeks ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 60368:

Twenty Fourteen: Fixes Latest Comment Link Color in Content Sidebar.

The white link text had worked against the dark background of the Primary and Footer widget areas, but not against the light background in the Content Sidebar. Therefore, r60368 excludes the Content Sidebar from that ruleset.

Props sabernhardt, dhruvang21, ravigadhiyawp, sandeepdahiya.
Fixes #63520.

Last edited 2 weeks ago by sabernhardt (previous) (diff)

#12 @sabernhardt
2 weeks 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.


12 days ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


12 days ago

#15 @audrasjb
12 days ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 60425:

Twenty Fourteen: Fixes Latest Comment Link Color in Content Sidebar.

The white link text had worked against the dark background of the Primary and Footer widget areas, but not against the light background in the Content Sidebar. Therefore, r60368 excludes the Content Sidebar from that ruleset.

Reviewed by audrasjb.
Merges [60368] to the 6.8 branch.
Props sabernhardt, dhruvang21, ravigadhiyawp, sandeepdahiya.
Fixes #63520.

Note: See TracTickets for help on using tickets.