Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#58713 closed defect (bug) (fixed)

Backport: Update default templates info

Reported by: ntsekouras Owned by: isabel_brison
Priority: normal Milestone: 6.3
Component: Editor Version:
Severity: normal Keywords: has-unit-tests
Cc: Focuses:

Description (last modified by ntsekouras)

Attachments (1)

58713.diff (1.3 KB ) - added by SergeyBiryukov 3 years ago.

Download all attachments as: .zip

Change History (11)

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


3 years ago
#1

  • Keywords has-patch has-unit-tests added

Trac ticket: https://core.trac.wordpress.org/ticket/58713

This PR back ports the changes from this Gutenberg PR: https://github.com/WordPress/gutenberg/pull/51428

#2 @ntsekouras
3 years ago

  • Description modified (diff)

#3 @isabel_brison
3 years ago

  • Owner set to isabel_brison
  • Resolutionfixed
  • Status newclosed

In 56145:

Editor: update template titles in browse mode.

Adds more meaningful titles to templates in the browse mode template list.

Props ntsekouras, audrasjb.
Fixes #58713.

@isabel_brison commented on PR #4793:


3 years ago
#4

Committed in r56145 / 9d141e5.

#5 @kebbet
3 years ago

  • Keywords needs-patch added; has-patch removed
  • Resolution fixed
  • Status closedreopened

The description for Author archives was updated in [56145] to :
Displays a single author\'s post archive. This template will serve as a fallback when a more a specific template (e.g., Author: Admin) cannot be found.

That seems to be incorrect, and should be something like this:
Displays a single author\'s post archive. This template will serve as a fallback when a more specific template (e.g., Author: Admin) cannot be found.

#6 @audrasjb
3 years ago

  • Resolutionfixed
  • Status reopenedclosed

Fixed by [56182] (forgot to fix the ticket directly in the commit).

#7 @audrasjb
3 years ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review6.3

@SergeyBiryukov
3 years ago

#8 @SergeyBiryukov
3 years ago

  • Resolution fixed
  • Status closedreopened

Thanks for the commits!

It appears that there is now a minor inconsistency in the descriptions:

  • Single Entries: Displays any single entry, such as a post or a page. This template will serve as a fallback when a more specific template (e.g., Single Post, Page, or Attachment) cannot be found.
  • Author Archives: This template will serve as a fallback when a more specific template (e.g., Author: Admin) cannot be found.
  • Category Archives: Displays a post category archive. This template will serve as a fallback when more specific template (e.g., Category: Recipes) cannot be found.
  • Tag Archives: Displays a post tag archive. This template will serve as a fallback when more specific template (e.g., Tag: Pizza) cannot be found.

Note the lack of the "a" article in the latter two strings. It should probably be reinstated, see 58713.diff.

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


3 years ago

#10 @isabel_brison
3 years ago

  • Resolutionfixed
  • Status reopenedclosed

In 56230:

Editor: fix typos in templates info.

Fixes two small typos in wp-includes/block-template-utils.php.

Props SergeyBiryukov.
Fixes #58713.

Note: See TracTickets for help on using tickets.