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
#1 2469231-popup_onload-menu-local-task.patch437 bytesjwilson3
popup_onload-local-task.png58.23 KBjwilson3
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 commented
Status: Active » Needs review
FileSize
2469231-popup_onload-menu-local-task.patch437 bytes

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
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark for HowlRound commented
jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark for HowlRound commented
Related issues: +#2469239: Proper English pluralization

Related followup: #2469239: Proper English pluralization

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark for HowlRound commented
Issue summary: View changes
FileSize
2469231-popup_onload-menu-local-task.patch642 bytes

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

ivnish’s picture

ivnish CreditAttribution: ivnish commented

This patch will be applied coming soon

  • itcrowd72 committed 4bdb2ff on 7.x-1.x authored by jwilson3 
    Issue #2469231 by jwilson3: Missing local task (tab) for /admin/content/...
ivnish’s picture

ivnish CreditAttribution: ivnish commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.