Page MenuHomePhabricator

[BUG] Share link menu item is not working on File pages
Closed, ResolvedPublic

Description

"Share link" in the menu is disabled on file pages - it does not bring up the sharing dialog when you tap on it - but the menu item appears nevertheless.

Steps:
Go to the MACS J1149 Lensed Star 1 article
Tap on the lead image
Go to the file page for the image
Hit the menu in the upper right

Expected:
Standard "Share sheet" to share link to the file page appears.

Actual:
"Share link" appears in the menu, but when a user taps on it no sharing dialog appears.

tempFileForShare_20180406-180338.jpg (1,080×2,220 px, 799 KB)

Event Timeline

Niedzielski raised the priority of this task from to Needs Triage.
Niedzielski updated the task description. (Show Details)
Niedzielski subscribed.

It's worth considering whether we want to even support sharing facts from file pages, considering that we expect almost no users to ever get to a file page. It might be worth simply disabling share a fact and defaulting to normal sharing on file pages to avoid this issue entirely.

Dbrant renamed this task from Share-A-Fact on file pages should use the file when it is an image to Disable share-a-fact on non-main-namespace pages..Aug 13 2015, 4:42 PM
Dbrant updated the task description. (Show Details)
Dbrant set Security to None.
Dbrant moved this task from Needs Triage to Product Backlog on the Wikipedia-Android-App board.

@LGoto: Which Android App related project should this task be in? Please associate this task to an active 'code project' if possible. Thanks!

Charlotte renamed this task from Disable share-a-fact on non-main-namespace pages. to [BUG] Disable sharing from menu on non-main-namespace pages.Apr 6 2018, 5:09 PM
Charlotte triaged this task as Low priority.
Charlotte updated the task description. (Show Details)
RHo renamed this task from [BUG] Disable sharing from menu on non-main-namespace pages to [BUG] Share link menu item is not working on File pages.May 21 2018, 7:06 PM
RHo updated the task description. (Show Details)

Updated task description to clarify it is about the Share link option in the File page overflow, not about Share-a-fact (also similar to T106610 , let's just leave the Share-a-fact on file pages as is)

Change 439787 had a related patch set uploaded (by Cooltey; owner: Cooltey):
[apps/android/wikipedia@master] Fix: Share link menu item is not working on File pages

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

Note for testing: the Add to reading list menu does not work either.

Change 439787 merged by jenkins-bot:
[apps/android/wikipedia@master] Fix: Share link menu item is not working on File pages

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

ABorbaWMF subscribed.

I am not seeing this fix on 2.7.237-alpha-2018-06-27

Still seeing Share Link

Screenshot_20180628-190702.png (1,080×1,920 px, 1 MB)

Hi @ABorbaWMF

Yes, the "Share Link" is expected to see on a File page.
The ticket was talking about the expected behavior that a "share sheet" dialog should be seen when tapping on the "Share Link".