Make WordPress Core

Opened 19 months ago

Closed 16 months ago

Last modified 16 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 19 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 17 months ago.

Download all attachments as: .zip

Change History (17)

@jeherve
19 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.


19 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
19 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
19 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
19 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
19 months ago

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

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

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


17 months ago

#7 @devmuhib
17 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 17 months ago by devmuhib (previous) (diff)

@dhrumilk
17 months ago

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


17 months ago

#9 follow-up: @audrasjb
16 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
16 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 16 months ago by jeherve (previous) (diff)

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


16 months ago

#12 @estelaris
16 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
16 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
16 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:


16 months ago
#15

Thanks for the PR! Committed in r56720.

Note: See TracTickets for help on using tickets.