Make WordPress Core

Opened 5 months ago

Closed 2 months ago

Last modified 2 months ago

#58820 closed enhancement (fixed)

Admin Bar: add link to Learn to WordPress menu

Reported by: jeherve's profile jeherve Owned by: audrasjb's profile audrasjb
Milestone: 6.4 Priority: normal
Severity: normal Version:
Component: Toolbar Keywords: has-patch commit
Focuses: Cc:

Description

Tutorials and lessons on learn.wordpress.org have become useful resources to many WordPress site owners. In #57785, we're discussing adding links to those resources inside a dashboard widget.

In this Trac ticket, I'd like to suggest adding a link to the site to the existing WordPress menu in the top admin bar. It would be a great addition to the menu, providing quick access to those resources from anywhere in your WordPress dashboard.

Attachments (2)

58820-wordpress-menu.png (51.0 KB) - added by jeherve 5 months ago.
Preview of the WordPress menu in the admin bar, with a new "Learn WordPress" link
58820.diff (1.1 KB) - added by dhrumilk 3 months ago.

Download all attachments as: .zip

Change History (17)

@jeherve
5 months ago

Preview of the WordPress menu in the admin bar, with a new "Learn WordPress" link

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


5 months ago
#1

Adding a link to WordPress.org' Learn section would be a great addition to the menu, providing quick access to those resources from anywhere in your WordPress dashboard.

#2 @mikinc860
5 months ago

This option looks good to me as it can help new learners direct from their dashboard. Here I have one small suggestion that the link should open to the new tab.

#3 @audrasjb
5 months ago

  • Milestone changed from Awaiting Review to 6.4
  • Owner set to audrasjb
  • Status changed from new to accepted

Thanks for the ticket and for the PR.
The proposed PR works fine. Moving to milestone 6.4 and self assigning for commit.

@mikinc860: except for very specific cases, opening a link in a new window/tab is bad. Please have a look at the related section of the Accessibility Handbook :)

#4 @sabernhardt
5 months ago

  • Component changed from Menus to Toolbar
  • Focuses docs removed

#58595 might move all those external links to a dedicated admin page so they have more context. It also could avoid surprising anyone by opening links in a new tab (or the same tab when expectations are different).

#5 @courane01
5 months ago

See ideas also on #57785, including the Slack discussion with Andrea.

Last edited 2 months ago by SergeyBiryukov (previous) (diff)

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


3 months ago

#7 @devmuhib
3 months ago

  • Keywords 2nd-opinion added

@courane01 Needs to move forward this ticket. In bug-scrub meeting we are trying to move this ticket forward.

Last edited 3 months ago by devmuhib (previous) (diff)

@dhrumilk
3 months ago

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


3 months ago

#9 follow-up: @audrasjb
2 months ago

  • Keywords changes-requested added; 2nd-opinion removed

Thanks for the patch, however AFAIK the Learn link doesn't seem to need any internationalization since people are being redirected automatically to their language.

#10 in reply to: ↑ 9 @jeherve
2 months ago

Replying to audrasjb:

Thanks for the patch, however AFAIK the Learn link doesn't seem to need any internationalization since people are being redirected automatically to their language.

👍 I refreshed my pull request to remove internationalization on the link. I had originally added it to match the format used for the other links in the admin bar, but I'm happy to leave it out if it's not necessary.

Noting that there is also conversation about this here:
https://wordpress.slack.com/archives/C059LE39BLG/p1689705781225109

Last edited 2 months ago by jeherve (previous) (diff)

This ticket was mentioned in Slack in #design by estelaris. View the logs.


2 months ago

#12 @estelaris
2 months ago

This is not the best solution for new users but should be considered as the best we can do before the admin redesign.

#13 @hellofromTonya
2 months ago

  • Keywords commit added; changes-requested removed

Changes requested were made in PR 4854 and approved by @audrasjb. Marking it for commit.

#14 @costdev
2 months ago

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

In 56720:

Toolbar: Link to Learn WordPress in the WordPress Logo menu.

This aims to make the various resources on learn.wordpress.org more easily available.

Props jeherve, mikinc860, audrasjb, sabernhardt, courane01, devmuhib, dhrumilk, estelaris, hellofromTonya.
Fixes #58820.

@costdev commented on PR #4854:


2 months ago
#15

Thanks for the PR! Committed in r56720.

Note: See TracTickets for help on using tickets.