Make WordPress Core

Opened 8 months ago

Closed 7 months ago

Last modified 6 months 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 8 months ago.
Before patch
after.png (158.2 KB) - added by dhruvang21 8 months ago.
before patch.png (65.7 KB) - added by sandeepdahiya 8 months ago.
Before applying the patch
after patch.png (70.0 KB) - added by sandeepdahiya 8 months ago.
After applying the patch

Download all attachments as: .zip

Change History (20)

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


8 months ago
#1

  • Keywords has-patch added

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

Trac 63520

@sabernhardt commented on PR #8888:


8 months 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
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

  1. ✅ Issue resolved with patch.

Supplemental Artifacts

Added as Attachment

@dhruvang21
8 months ago

Before patch

@dhruvang21
8 months ago

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

  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 8 months ago by ravigadhiyawp (previous) (diff)

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

  1. ✅ Issue resolved with patch.

Additional Notes

  • This css patch is working fine.

Supplemental Artifacts

Screenshot: https://prnt.sc/uhu0QTAGmpna

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

@sandeepdahiya
8 months ago

Before applying the patch

@sandeepdahiya
8 months ago

After applying the patch

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

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

#10 @karmatosed
7 months ago

  • Milestone changed from 6.9 to 6.8.2

#11 @karmatosed
7 months ago

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

In 60368:

Twenty Twenty-Four: Fixes Latest Comment Link Color in Sidebar.

This sets the link color to work for dark areas when 51205 set to white for any widget area. This was not good for Primary and Footer Widget Areas.

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

Version 4, edited 7 months ago by sabernhardt (previous) (next) (diff)

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

#15 @audrasjb
7 months 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.

@sabernhardt commented on PR #8888:


6 months ago
#16

Committed in r60368

Note: See TracTickets for help on using tickets.