Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#58597 closed enhancement (fixed)

Allow non-block themes to add theme support for link color settings

Reported by: poena's profile poena Owned by: audrasjb's profile audrasjb
Milestone: 6.3 Priority: normal
Severity: normal Version:
Component: Themes Keywords: has-patch has-screenshots commit needs-docs has-dev-note
Focuses: Cc:

Description (last modified by poena)

Allow themes without theme.json to opt-in to block link color tools via:

add_theme_support( 'link-color' );

Follow up to https://core.trac.wordpress.org/ticket/57460 and https://core.trac.wordpress.org/ticket/57649 with a more limited scope than 'appearance-tools'.

Corresponding Gutenberg PR: https://github.com/WordPress/gutenberg/pull/51775

Attachments (3)

58597.diff (736 bytes) - added by poena 2 years ago.
Updates theme support in wp-includes/class-wp-theme-json-resolver.php
Capture d’écran 2023-06-22 à 13.43.16.png (157.8 KB) - added by audrasjb 2 years ago.
Before patch: no support for link's custom styles
Capture d’écran 2023-06-22 à 13.43.34.png (204.6 KB) - added by audrasjb 2 years ago.
After patch: wooohooo \o/

Download all attachments as: .zip

Change History (11)

@poena
2 years ago

Updates theme support in wp-includes/class-wp-theme-json-resolver.php

#1 @poena
2 years ago

Testing instructions

Activate a classic theme.

Open the themes functions.php and locate a setup function or similar that uses
add_action( 'after_setup_theme'. For example, if you are using Twenty Twenty-One, look for the function called
twenty_twenty_one_setup()

Add add_theme_support( 'link-color' ); inside that function.
Open the block editor.
Add a paragraph with a link.
Open the block settings sidebar and confirm that the link option is available in the color panel.

Select a custom link color using the color picker. Confirm that the color works in the editor and the front.

(I am suggesting the color picker because themes need to add their own CSS to make the custom theme palette colors work)

#2 @poena
2 years ago

  • Description modified (diff)

#3 @poena
2 years ago

  • Component changed from General to Themes
  • Keywords has-patch added

@audrasjb
2 years ago

Before patch: no support for link's custom styles

@audrasjb
2 years ago

After patch: wooohooo \o/

#4 @audrasjb
2 years ago

  • Keywords has-screenshots commit added

Thanks for the ticket and patch Carolina!

Tested on my side using a classic theme: works fine!

Self assigning for commit.

#5 @audrasjb
2 years ago

  • Keywords needs-dev-note needs-docs added

#6 @audrasjb
2 years ago

  • Owner set to audrasjb
  • Status changed from new to accepted

#7 @audrasjb
2 years ago

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

In 55996:

Themes: Allow non-block themes to add theme support for link color settings.

This changeset themes without theme.json to opt-in to block link color tools via add_theme_support( 'link-color' );.

Follow-up to [55067], [55273].

Props poena, audrasjb.
Fixes #58597.

#8 @stevenlinx
2 years ago

  • Keywords has-dev-note added; needs-dev-note removed
Note: See TracTickets for help on using tickets.