Hey @JTannerWMF, please review – and move to "Needs analytics" or "Ready for dev" afterwards.

PS: Consider that the tooltips must contain the actual name of the list item ("Places" and "View on map") to ensure that users find them after encountering the tooltips

Thanks!

@JTannerWMF moved this to Ready for dev for now. And still feel free to review it.

@JTannerWMF @scblr

This is still work in progress, but wanted to settle the ordering of tooltips.

Here is the apk if you want to test it out: https://github.com/wikimedia/apps-android-wikipedia/pull/4407/checks

I made a demo too for a fresh install: https://www.youtube.com/watch?v=dPoOmY2twY8

Some questions:
@scblr : I kept the position of the tooltip consistent with other tooltips on article screen - to be shown at the bottom of the anchor. Hope that is ok. Also since the copy changed, the line break is now at a different place. Please lmk if that is ok.

@JTannerWMF and @scblr

1. On the main nav tab the tooltips do not clash at all, which is great. I have just prioritized the places tooltip over edit tab tooltip.
2. Article screen has 3 tooltips: Customize toolbar, Watchlist and now Places. First things first, can we remove any :) if not, does the order in the video - Places, watchlist[Depends on when the login happens - as seen in video if the user logs in after seeing the customize tooltip the order varies], customize toolbar look good? Any other logic among them?

Some questions:
@scblr : I kept the position of the tooltip consistent with other tooltips on article screen - to be shown at the bottom of the anchor. Hope that is ok. Also since the copy changed, the line break is now at a different place. Please lmk if that is ok.

Yes, that’s fine.

@JTannerWMF and @scblr

1. On the main nav tab the tooltips do not clash at all, which is great. I have just prioritized the places tooltip over edit tab tooltip.

Sounds good!

2. Article screen has 3 tooltips: Customize toolbar, Watchlist and now Places. First things first, can we remove any :) if not, does the order in the video - Places, watchlist[Depends on when the login happens - as seen in video if the user logs in after seeing the customize tooltip the order varies], customize toolbar look good? Any other logic among them?

I’ll sync with @JTannerWMF later today and will follow up.

Thank you for the review @scblr . Will update this once you have had a chance to discuss this and update here.

Thanks @Sharvaniharan

Article screen has 3 tooltips: Customize toolbar, Watchlist and now Places. First things first, can we remove any :)

I synced with JT. We’d like to eliminate the Watchlist tooltip in the article AND on the main screen. So let’s say goodbye to these two:

if not, does the order in the video - Places, watchlist[Depends on when the login happens - as seen in video if the user logs in after seeing the customize tooltip the order varies], customize toolbar look good? Any other logic among them?

Two important things:

• The "Customize toolbar" (first launch of the app) tooltip should be shown before "View on map" (earliest on second launch of the app)
• The "View on map" tooltip should only be shown when the article contains location information. In the video you posted, it is shown for the Nyctibatrachus major article, which does not contain location information:

Hi @scblr
Thank you for the update. This is ready for review.
APK: https://github.com/wikimedia/apps-android-wikipedia/pull/4407/checks

One thing I want to draw your attention to - The main menu tooltip is the first thing that new users will see on a fresh installation. Hope that is ok.

Hi @scblr

Update: Synced with @JTannerWMF about the main nav tooltip and she wants it to be displayed on second launch of the app, so I have added that changes as well.

Looks good to me @Sharvaniharan

Working for me on 2.7.50470-beta-2024-02-08