Change 722730 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Begin using core menu definitions for sourcing menu data

https://gerrit.wikimedia.org/r/722730

I think it makes sense to work towards deprecating the MobileMenu hook, so that onSkinTemplateNavigation::Universal and onSidebarBeforeOutput can become the standard way for extensions to modify menus. MobileMenu also doesn't seem super well documented, so moving forward with a consistent approach seems good to me!

Thanks for the sanity check!

Change 722729 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/GrowthExperiments@master] Update menu items via hook

https://gerrit.wikimedia.org/r/722729

Change 722729 abandoned by Jdlrobson:

[mediawiki/extensions/GrowthExperiments@master] Update menu items via hook

Reason:

Not working on this right now. Will pick this up again soon. Please let me know if there is any problems with working towards this from the Growth Team side. We don't want to have mobile specific hooks for menu modification on the longer term.

https://gerrit.wikimedia.org/r/722729

Example (To port the menu that is shown to logged in AMC users:)

diff --git a/includes/Menu/Entries/SingleMenuEntry.php b/includes/Menu/Entries/SingleMenuEntry.php
index 9ab4e0d45..951e068a8 100644
--- a/includes/Menu/Entries/SingleMenuEntry.php
+++ b/includes/Menu/Entries/SingleMenuEntry.php
@@ -44,8 +44,9 @@ class SingleMenuEntry implements IMenuEntry {
 	 * @param string $text Text to show on menu element
 	 * @param string $url URL menu element points to
 	 * @param string $className Additional CSS class names.
+	 * @param string|null $icon icon to use
 	 */
-	public function __construct( $name, $text, $url, $className = '' ) {
+	public function __construct( $name, $text, $url, $className = '', $icon = null ) {
 		$this->name = $name;
 		if ( $className ) {
 			$className .= ' ';
@@ -53,7 +54,7 @@ class SingleMenuEntry implements IMenuEntry {
 		$className .= 'menu__item--' . $name;
 
 		$this->attributes = [
-			'icon' => null,
+			'icon' => $icon,
 			'text' => $text,
 			'href' => $url,
 			'class' => $className
diff --git a/includes/Menu/Main/DefaultMainMenuBuilder.php b/includes/Menu/Main/DefaultMainMenuBuilder.php
index 7bf529762..7eaeb8358 100644
--- a/includes/Menu/Main/DefaultMainMenuBuilder.php
+++ b/includes/Menu/Main/DefaultMainMenuBuilder.php
@@ -76,7 +76,6 @@ final class DefaultMainMenuBuilder implements IMainMenuBuilder {
 	public function getGroups(): array {
 		$donate = $this->showDonateLink ?
 			BuilderUtil::getDonateGroup( $this->definitions ) : null;
-
 		$groups = [
 			BuilderUtil::getDiscoveryTools( $this->definitions ),
 			$this->getPersonalTools( $this->showMobileOptions ),
diff --git a/includes/Menu/User/AdvancedUserMenuBuilder.php b/includes/Menu/User/AdvancedUserMenuBuilder.php
index 8f8089812..0b38bd895 100644
--- a/includes/Menu/User/AdvancedUserMenuBuilder.php
+++ b/includes/Menu/User/AdvancedUserMenuBuilder.php
@@ -67,31 +67,26 @@ final class AdvancedUserMenuBuilder implements IUserMenuBuilder {
 	 */
 	public function getGroup( array $personalTools ): Group {
 		$group = new Group( 'p-personal' );
-		$group->insertEntry( new ProfileMenuEntry( $this->user ) );
-		$talkPage = $this->user->getUserPage()->getTalkPageIfDefined();
-		if ( $talkPage ) {
-			$entry = SingleMenuEntry::create(
-				'userTalk',
-				$this->messageLocalizer->msg( 'mobile-frontend-user-page-talk' )->escaped(),
-				$talkPage->getLocalURL()
-			);
-			$group->insertEntry( $entry );
-		}
-		$sandbox = $personalTools['sandbox']['links'][0] ?? false;
-
-		if ( $sandbox ) {
-			$group->insertEntry( SingleMenuEntry::create(
-				'sandbox',
-				$sandbox['text'],
-				$sandbox['href']
-			) );
-		}
-		$this->definitions->insertWatchlistMenuItem( $group );
-		$this->definitions->insertContributionsMenuItem( $group );
-		if ( $this->user->isRegistered() ) {
-			$this->definitions->insertLogOutMenuItem( $group );
-		} else {
-			$this->definitions->insertLogInMenuItem( $group );
+		unset( $personalTools[ 'preferences' ] );
+		unset( $personalTools[ 'betafeatures' ] );
+		unset( $personalTools[ 'uploads' ] );
+		foreach ( $personalTools as $key => $definition ) {
+			if ( $definition['links'] ) {
+				$link = $personalTools[$key]['links'][0] ?? false;
+				if ( $link ) {
+					$class = $link['class'] ?? '';
+					if ( is_array( $class ) ) {
+						$class = implode( ' ', $class );
+					}
+					$group->insertEntry( SingleMenuEntry::create(
+						$key,
+						$link['text'],
+						$link['href'],
+						$class,
+						$link['icon'] ?? null
+					) );
+				}
+			}
 		}
 		Hooks::run( 'MobileMenu', [ 'user', &$group ] );
 		return $group;

Targetting 1.38 as a stretch goal.
https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/753596 will allow Extension:GrowthExperiments to use the SkinTemplateNavigation::Universal hook to make modifications to menu items when operating in mobile mode.

Will coordinate with Growth team if we make progress on our side.

Change 753596 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] [Refactor] Use core definitions for personal menu icons

https://gerrit.wikimedia.org/r/753596

Change 755046 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Main page definition come from MediaWiki:Sidebar

https://gerrit.wikimedia.org/r/755046

Change 755047 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/extensions/GrowthExperiments@master] Drop usage of MobileMenu hook for user page

https://gerrit.wikimedia.org/r/755047

Change 755048 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Deprecate MobileMenu hook

https://gerrit.wikimedia.org/r/755048

Change 753596 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] [Refactor] Use core definitions for personal menu icons

https://gerrit.wikimedia.org/r/753596

Change 756021 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/core@master] Add icons to navigation sidebar entries

https://gerrit.wikimedia.org/r/756021

Change 756021 merged by jenkins-bot:

[mediawiki/core@master] Add icons to navigation sidebar entries

https://gerrit.wikimedia.org/r/756021

Change 755046 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Main page definition come from MediaWiki:Sidebar

https://gerrit.wikimedia.org/r/755046

In T291568#7673281, @Jdlrobson wrote:

@kostajh @Tgr requesting review on https://gerrit.wikimedia.org/r/c/722729 so I can deprecate the MobileMenu hook in time for the 1.38 release. Thanks in advance. Let me know if that timeline is not realistic.

Yes, we should be able to get to it in time – https://www.mediawiki.org/wiki/MediaWiki_1.38 says "spring 2022", so sometime in the next few weeks?

so sometime in the next few weeks?

That would be perfect. Thanks.

Change 763306 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/MinervaNeue@master] Allow text to be overridable

https://gerrit.wikimedia.org/r/763306

Change 763306 abandoned by Jdlrobson:

[mediawiki/skins/MinervaNeue@master] Allow text to be overridable

Reason:

https://gerrit.wikimedia.org/r/763306

Change 763306 restored by Jdlrobson:

[mediawiki/skins/MinervaNeue@master] Allow text to be overridable

https://gerrit.wikimedia.org/r/763306

Change 763306 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Allow msg key to be overridable

https://gerrit.wikimedia.org/r/763306

Change 755047 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Drop usage of MobileFrontend MobileMenu hook in favor of core hooks

https://gerrit.wikimedia.org/r/755047

Change 764416 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/core@master] skins: Fix Skin::buildSidebar to not share cache between skins

https://gerrit.wikimedia.org/r/764416

Change 755048 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Deprecate MobileMenu hook

https://gerrit.wikimedia.org/r/755048

Moved this to blocked as I still see one outstanding patch associated with this ticket: https://gerrit.wikimedia.org/r/c/mediawiki/core/+/764416 that I believe @Jdlrobson is reviewing (please correct if wrong)