openstreetmap-website icon indicating copy to clipboard operation
openstreetmap-website copied to clipboard

Avoid suggesting to add OSM to home screen in the login flow

Open HarelM opened this issue 3 months ago • 9 comments

Problem

When using OSM as a third party OAuth popup, OSM suggest to add itself to the home screen on Android devices.

Description

It would be great to avoid this suggestion in the OAuth login flow in order to avoid confusing the users of the original app/website. Israel hiking map, in our case.

  • I believe it is related to the following feature if I needed to guess: #2361

CC: @1ec5

Screenshots

Image

HarelM avatar Sep 21 '25 18:09 HarelM

Why do you think it's a related to a ticket that was closed with no action taken?

tomhughes avatar Sep 21 '25 19:09 tomhughes

One of the "benefits" of PWA is usually this pop up, that allows you to add a website to the home screen. But I might wrong... I can edit this part if it helps, this is not the main point of this issue, obliviously...

HarelM avatar Sep 21 '25 20:09 HarelM

Pretty sure we aren’t doing anything specifically to enable installation as an app on Android. We are setting apple-touch-icon and msapplication-TileImage meta tags, but not google-play-app.

1ec5 avatar Sep 21 '25 21:09 1ec5

Pretty sure we aren’t doing anything specifically to enable installation as an app on Android. We are setting apple-touch-icon and msapplication-TileImage meta tags, but not google-play-app.

PWA does not relate to google-play-app or any other native app.

HolgerJeromin avatar Sep 22 '25 06:09 HolgerJeromin

Yes, and we also don’t provide a PWA manifest or anything like that. @HarelM, could it be something to do with how you’ve implemented the login flow on your end?

1ec5 avatar Sep 22 '25 06:09 1ec5

Please stop fixating on the word WPA, I made a mistake, please read the issue description. There's nothing special about the login flow on my end, I simply open a window/tab: https://github.com/IsraelHikingMap/Site/blob/0f8efa6441274bd7e093d481d17df9c8ca359163/IsraelHiking.Web/src/application/services/authorization.service.ts#L40

HarelM avatar Sep 22 '25 06:09 HarelM

This page says, it's the manifest file:

<link rel="manifest" href="/assets/manifest-e89323590a852a776c60b03fe9cd7bae2ad4dc3fe624dff32bc77899e4c12ba3.json">

mmd-osm avatar Sep 22 '25 14:09 mmd-osm

Ack you’re right, I missed that when searching the source code of the homepage on my phone last night. 😅 So then this comes from #1558, which we also used for a geo: URI protocol handler in #5736…

1ec5 avatar Sep 22 '25 16:09 1ec5

According to the statement here: https://github.com/openstreetmap/openstreetmap-website/pull/1558#issuecomment-306233409 It seems that adding to home screen existed before #1558, but I'm guessing @mmd-osm's comment is the relevant one in terms of when this was added...

HarelM avatar Sep 23 '25 10:09 HarelM