WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 2 months ago

#53556 closed enhancement (fixed)

Add a new template-editing tag to the Theme API

Reported by: poena Owned by: SergeyBiryukov
Milestone: 5.8.1 Priority: normal
Severity: normal Version:
Component: Themes Keywords: has-patch commit i18n-change fixed-major
Focuses: Cc:

Description

With the template editing being a new feature in 5.8 that is opt-in,
a new tag should be added so that users can filter and search for themes that support it.

Suggest tag: template-editing
Label: Template editing
Description: Support template editing in the block editor

Attachments (3)

add-theme.png (138.8 KB) - added by desrosj 3 months ago.
53556.patch (520 bytes) - added by poena 2 months ago.
53556.diff (954 bytes) - added by desrosj 2 months ago.

Download all attachments as: .zip

Change History (13)

@desrosj
3 months ago

#1 @desrosj
3 months ago

  • Keywords reporter-feedback added

Just to clarify, this request is to add the Template Editing feature to this screen?

I don't think anything would prevent a theme author from adding that tag to their theme's header as I can't find any code that validates the list of tags for a theme.

I'm also not sure if a corresponding meta ticket would be needed to return that tag in the API results. I'd need to do some testing.

#2 @SergeyBiryukov
3 months ago

  • Milestone changed from Awaiting Review to 5.8.1

The corresponding Meta ticket is #meta5802.

Core keeps its own list of these tags in get_theme_feature_list(), so we can add the new one there as well.

Related: #50164, #50165.

#3 @poena
3 months ago

Yes that is correct, I am suggesting that template editing is added to the feature list.

When a theme is submitted, it goes through an automated check to see if it is using the correct tags.
Until two weeks ago, this check did block themes that had incorrect tags.

#4 @SergeyBiryukov
3 months ago

  • Keywords needs-patch added; reporter-feedback removed

@poena
2 months ago

#5 @poena
2 months ago

  • Keywords has-patch added; needs-patch removed

@desrosj
2 months ago

#6 @desrosj
2 months ago

  • Keywords commit i18n-change added

53556.diff adds a @since tag to note when Template Editing was added as a feature.

I've marked as i8n-change as the Polyglot team would need to be made aware of this change should it be backported to 5.8.1. I think it makes sense to include it, but the #meta5802 ticket should ideally land at the same time.

Marking commit so that @SergeyBiryukov can shepherd both of these in within a reasonable amount of time of each other.

This ticket was mentioned in Slack in #themereview by poena. View the logs.


2 months ago

#8 @SergeyBiryukov
2 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 51510:

Themes: Add "Template Editing" to the list of WordPress theme features.

Follow-up to [meta11158].

Props poena, desrosj.
Fixes #53556. See #meta5802.

#9 @SergeyBiryukov
2 months ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening for backporting to the 5.8 branch.

#10 @desrosj
2 months ago

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

In 51516:

Themes: Add "Template Editing" to the list of WordPress theme features.

Follow-up to [meta11158].

Props poena, desrosj, SergeyBiryukov.
Merges [51510] to the 5.8 branch.
Fixes #53556. See #meta5802.

Note: See TracTickets for help on using tickets.