On vanilla Drupal 7, with the normal Toolbar module, if you navigate to /admin/content there is no way to know that the /popup_onload page exists.

This bug is mentioned several times in the comments on the tutorial page: http://smartwolverine.net/article/popup-load-tutorial-part-1

Someone (maybe maintainer -- not sure) said they would create a ticket for this, but I couldn't find it.

The two workarounds seem to be to either add a shortcut, or to use admin menu.

But I would much prefer to have a visible tab, to follow Drupal conventions, such as with this screenshot:

popup_onload-menu-task

Here comes a patch that fixes this UX nightmare and will make our clients very happy ;)

The latest patch incorporates changes from #2469239: Proper English pluralization to modify the confusing tab text from "Popup On Loads" to simply "Popups".

CommentFileSizeAuthor
#4 2469231-popup_onload-menu-local-task.patch642 bytesjwilson3
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View
#1 2469231-popup_onload-menu-local-task.patch437 bytesjwilson3
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View
popup_onload-local-task.png58.23 KBjwilson3
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

jwilson3’s picture

Status: Active » Needs review
FileSize
437 bytes
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View

This patch uses hook_menu_alter() to add the menu local task because the menu path 'admin/content/popup_onload' is added via the entity system and not in hook_menu.

jwilson3’s picture

jwilson3’s picture

jwilson3’s picture

Issue summary: View changes
FileSize
642 bytes
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View

The latest patch incorporates changes from #2469239: Proper English pluralization to modify the confusing tab text from "Popup On Loads" to simply "Popups".