as per the title, patch coming

CommentFileSizeAuthor
#4 2857484-4-admin-setting-default-menu-tab.patch1.14 KBsupriya1992
#2 biblio-2857484-2-admin-settings-default-menu-tab.patch1.25 KBAlan D.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Alan D. created an issue. See original summary.

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Status: Active » Needs review
FileSize
biblio-2857484-2-admin-settings-default-menu-tab.patch1.25 KB

This makes the default tab active when clicking the parent item, plus less code :)

Status: Needs review » Needs work

The last submitted patch, 2: biblio-2857484-2-admin-settings-default-menu-tab.patch, failed testing. View results

supriya1992’s picture

supriya1992 CreditAttribution: supriya1992 at Valuebound for Valuebound commented
FileSize
2857484-4-admin-setting-default-menu-tab.patch1.14 KB

I have created patch. I hope it will apply successfully.

supriya1992’s picture

supriya1992 CreditAttribution: supriya1992 at Valuebound for Valuebound commented
Status: Needs work » Needs review
Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented

That was a simple reroll by supriya1992. There was a single newline change between the two patches.

  • Liam Morland committed bfb0fee on 7.x-1.x authored by Alan D. 
    Issue #2857484 by Alan D., supriya1992: Use MENU_DEFAULT_LOCAL_TASK for...
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Status: Needs review » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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