Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#54707 closed enhancement (fixed)

Changing the title of the page for the Themes screen and added a link to the documentation

Reported by: jdy68's profile jdy68 Owned by: audrasjb's profile audrasjb
Milestone: 6.0 Priority: normal
Severity: normal Version:
Component: Help/About Keywords: has-patch commit
Focuses: docs Cc:

Description

The title of the "Themes" screen page is "Manage Themes" and does not match the title of the screen which is "Themes" and this is the only screen in the WordPress dashboard where there is a difference between the screen name and the page title.
See: current-page-title-of-theme-screen.png

To fix this error, just modify a line in the file /wp-admin/themes.php.
See result: modified-page-title-of-theme-screen.png

A link to the documentation describing the “Themes” screen is missing.
To add this link, just add a line in the "wp-admin/themes.php" file
See results: adding-link-to-help-tab.png

Attachments (4)

current-page-title-of-theme-screen.png (13.2 KB) - added by jdy68 2 years ago.
modified-page-title-of-theme-screen.png (13.5 KB) - added by jdy68 2 years ago.
adding-link-to-help-tab.png (38.3 KB) - added by jdy68 2 years ago.
54707.diff (1004 bytes) - added by jdy68 2 years ago.

Download all attachments as: .zip

Change History (12)

@jdy68
2 years ago

#1 @jdy68
2 years ago

  • Keywords has-patch added

#2 @audrasjb
2 years ago

  • Component changed from Administration to Help/About
  • Focuses docs added
  • Keywords commit added
  • Milestone changed from Awaiting Review to 6.0
  • Owner set to audrasjb
  • Status changed from new to accepted

The patch looks good to me and this change makes sense, thank you!
Moving this to milestone 6.0 since 5.9 is in soft string freeze state.

#4 @audrasjb
2 years ago

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

In 52543:

Help/About: Change the HTML title of Themes Screen and add related HelpHub Docs link.

This change replaces the title of the Appearance Themes Screen to make it consistent with the main heading of the screen and with other Admin screens. It also adds a link to the related HelpHub Documentation in the Help tab of the screen.

Props jdy68.
Fixes #54707.

#6 @SergeyBiryukov
2 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This looks good, but I think the "Documentation on the Appearance Themes Screen" link could probably be changed to "Documentation on Managing Themes" from 54707.diff.

That would be more consistent with similar links on other screens:

  • "Documentation on Managing Posts"
  • "Documentation on Managing Pages"
  • "Documentation on Managing Links"
  • "Documentation on Managing Plugins"
  • "Documentation on Managing Users"

#7 @audrasjb
2 years ago

Alright. Makes sense 👍

#8 @audrasjb
2 years ago

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

In 52545:

Help/About: Correction in "Managing Themes" HelpHub Docs link.

This change updates the HelpHub Documentation link to "Managing Themes" to make it more consistent with other Help tabs.

Props SergeyBiryukov, jdy68.
Fixes #54707.

Note: See TracTickets for help on using tickets.