Page MenuHomePhabricator
Create Task
Maniphest T273797

Talk page diff links don't play nicely with Android intents / open-with dialog
Open, Needs TriagePublic
Actions

Description

Diffs to talk pages (such as users receive in email watchlist notifications) result in a loop when you try to open them with the Android app.

The new native Talk page display is great! I can follow a talk link from a watchlist email and see what someone said! It looks great!

However, if I open the diff link from such a watchlist email, it opens the 'open with' dialog, and if I choose the Android app, it immediately re-opens the 'open with' dialog, and I can engage in an endless cycle of trying and failing to view a diff using the app.

Related Objects

Event Timeline

Ragesoss created this task.Feb 3 2021, 5:52 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 3 2021, 5:52 PM
Ragesoss added a parent task: T273253: [EPIC] POST V2.7.33X-R-Rosgulla Initial Talk Pages Release Clean Up.Feb 3 2021, 5:52 PM
Kizule added a project: Wikipedia-Android-App.Feb 3 2021, 6:06 PM
Dbrant added a comment.Feb 3 2021, 6:21 PM

Thanks for checking it out, @Ragesoss !
We're definitely thinking about how to handle this better. It happens because the app currently doesn't handle user contribution pages natively, so it bounces the user out to the user's default web browser app. However, our app is also configured to handle any external links of the form *.wikipedia.org/wiki/*, so our app will also keep appearing in the list of "external" apps that can handle the link.
Unfortunately Android intents don't offer enough granularity to exclude URLs in a certain namespace (i.e. Special:) while including all the others.

LGoto subscribed.Feb 8 2021, 5:14 PM

related to https://phabricator.wikimedia.org/T120315

LGoto removed a parent task: T273253: [EPIC] POST V2.7.33X-R-Rosgulla Initial Talk Pages Release Clean Up.Feb 8 2021, 5:15 PM
LGoto moved this task from Needs Triage to Garage on the Wikipedia-Android-App board.
JTannerWMF subscribed.Feb 8 2021, 5:45 PM

In our Board Refinement meeting we discussed converting to webview in instances that we take users out of the app, once we do that it should fix this case.

Ragesoss added a comment.Feb 8 2021, 5:49 PM
In T273797#6812060, @JTannerWMF wrote:

In our Board Refinement meeting we discussed converting to webview in instances that we take users out of the app, once we do that it should fix this case.

From my perspective as a frequent user of the app, I think that would be a very nice improvement.

• scblr awarded a token.Feb 9 2021, 10:47 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits