Page MenuHomePhabricator

When I force press an image, the preview should be the image, and the share action should be available
Closed, ResolvedPublic

Description

On the article view, when I force press an image:

  • The preview should not show the share and close actions, only the image, caption and licese
  • I should be able to access the Share action via the peek actions (currently there are no peek actions)

Event Timeline

On the article view, when I long press a link:

What do you mean by "a link" - a link to an image? an image inline in an article? both?

On the gallery view
The preview should not show only the image (currently there is no peek)
I should be able to access the Share action via the peek actions (currently there are no peek actions)

So you want to be able to peek by 3d touching the current visible image in the gallery? Or do you mean the long press menu that currently only shows copy? If you do mean peek by 3d touching, is there precedent for other apps that peek that don't pop? or would pop go to another gallery?

JMinor renamed this task from When I long press an image, the preview should be the image, and the share action should be available to When I force press an image, the preview should be the image, and the share action should be available.Sep 11 2017, 6:19 PM
JMinor updated the task description. (Show Details)

@NHarateh_WMF unfortunately I'm still seeing the close and share buttons on 5.7.0 (1232) and although the peek action is appearing on scroll up, tapping on 'Share' closes the preview without opening the share sheet.

IMG_A5B928438053-1.jpeg (1×750 px, 1 MB)

@cmadeo what device and OS?

It looks like 5.7 doesn't have the latest changes, develop does. Will move this to Waiting for Build for now.

@NHarateh_WMF Sorry it's appearing correctly on 5.7.0 (1234) for me on an iPhone 7 running iOS 11.1

@NHarateh_WMF Would you like me to move this over to needs QA?

Testing on iPhone 6s (iOS 10.3.3) and Wikipedia app 5.7.0 (1235). This is fixed as shown in the first image there is no Share button when peeking, Share only appears after swiping up on the image.

T174714 (Force Press an Image).PNG (1×750 px, 1 MB)
T174714 (Swipe the image).PNG (1×750 px, 1 MB)

ABorbaWMF subscribed.

Testing on 5.7.0 (1236)

This is working for the lead image, but the share and close buttons are still present on subsequent images.

IMG_1607.jpg (2×1 px, 382 KB)

IMG_1608.jpg (2×1 px, 228 KB)

Looks good on lead images and article images! Thanks!