Places: Add View on map option to customize toolbar

[cooltey]

https://phabricator.wikimedia.org/T351394

In this PR:

• Fixed an issue where we did not update the icon status of "Edit article" correctly.
• Optimize some code related to the overflow menu and action tab.
• Adjusted the width of the overflow menu.

[sharvaniharan]

Hi @cooltey

When I click on View on map I see that it just loads the map to a general locations with a zoomed-out approach like this: https://youtube.com/shorts/0usNNFKtGVg
I was expecting that we will zoom to that article on the map.. is that not the case?

[cooltey]

Hi @cooltey

When I click on View on map I see that it just loads the map to a general locations with a zoomed-out approach like this: https://youtube.com/shorts/0usNNFKtGVg I was expecting that we will zoom to that article on the map.. is that not the case?

Done!

[sharvaniharan]

@cooltey I think the recent change opens up the bottom sheet on loading the map, with markers on the background. I was thinking of a more smoother experience like this:

• We get the location from the article
• When user clicks on View on map it takes them to the places fragment
• Here let us pass on that LatLng, and zoom slowly to that marker at that location, similar to the cluster experience
• And, let us not open the bottom sheet, which is actually looking a bit more disruptive to me.
  I feel this experience will be what they expect. Please let me know your thoughts.

[cooltey]

@cooltey I think the recent change opens up the bottom sheet on loading the map, with markers on the background. I was thinking of a more smoother experience like this:

* We get the location from the article

* When user clicks on `View on map` it takes them to the places fragment

* Here let us pass on that `LatLng`, and zoom slowly to that `marker` at that location, similar to the cluster experience

* And, let us not open the bottom sheet, which is actually looking a bit more disruptive to me.
  I feel this experience will be what they expect. Please let me know your thoughts.

Sounds good to me, I can remove the open-up logic from the PR. But let me hold this PR until we merge the cluster PR to the feature branch 😅

[cooltey]

@cooltey I think the recent change opens up the bottom sheet on loading the map, with markers on the background. I was thinking of a more smoother experience like this:

* We get the location from the article

* When user clicks on `View on map` it takes them to the places fragment

* Here let us pass on that `LatLng`, and zoom slowly to that `marker` at that location, similar to the cluster experience

* And, let us not open the bottom sheet, which is actually looking a bit more disruptive to me.
  I feel this experience will be what they expect. Please let me know your thoughts.

Done. Now it will not show the bottom sheet and also will zoom in to the max level.