Make WordPress Core

Opened 15 months ago

Closed 14 months ago

Last modified 13 months ago

#63520 closed defect (bug) (fixed)

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

Reported by: sabernhardt Owned by: karmatosed
Priority: normal Milestone: 6.8.2
Component: Bundled Theme Version: 5.8
Severity: normal Keywords: has-patch has-test-info
Cc: Focuses: css

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


15 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:


15 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
15 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
15 months ago

Before patch

@dhruvang21
15 months ago

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

Version 0, edited 15 months ago by ravigadhiyawp (next)

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

  • Milestone Awaiting Review6.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
15 months ago

Before applying the patch

@sandeepdahiya
15 months ago

After applying the patch

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


15 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
15 months ago

  • Keywords commit added
  • Owner set to karmatosed
  • Status newassigned

Assigning to myself to follow through and see about commit.

#10 @karmatosed
15 months ago

  • Milestone 6.96.8.2

#11 @karmatosed
15 months ago

  • Resolutionfixed
  • Status assignedclosed

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 14 months ago by sabernhardt (previous) (diff)

#12 @sabernhardt
14 months ago

  • Keywords commit removed
  • Resolution fixed
  • Status closedreopened

Reopening for backport consideration.

This ticket was mentioned in Slack in #core-themes by sabernhardt. View the logs.


14 months ago

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


14 months ago

#15 @audrasjb
14 months ago

  • Resolutionfixed
  • Status reopenedclosed

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:


13 months ago
#16

Committed in r60368

Note: See TracTickets for help on using tickets.