Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3D touching a link can lock the app #1818

Closed
manuroe opened this issue Mar 19, 2018 · 2 comments
Closed

3D touching a link can lock the app #1818

manuroe opened this issue Mar 19, 2018 · 2 comments
Labels
P2 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems UX
Milestone

Comments

@manuroe
Copy link
Member

manuroe commented Mar 19, 2018

Sometimes, 3D touching a link in a room can lock the app. The user needs to kill the app to use it again.

This happens at least in 2 manners:

  • The peek view stays displayed. Stopping pressing should make the peek view go away but it sometimes stays on the top of screen with no way to come back to the app.
  • After the pop (the user deeply pressed the link so that it has been opened in Safari), when the user reopens the app, the app displayed a blurry image of itself with no way to come back to a normal usage.

Peek and Pop doc:
https://developer.apple.com/library/content/documentation/UserExperience/Conceptual/Adopting3DTouchOniPhone/index.html#//apple_ref/doc/uid/TP40016543-CH1-SW1

Screenshots:
img_9622
img_9621

@manuroe manuroe added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems P2 UX labels Mar 19, 2018
@manuroe manuroe added this to the Sprint 9 milestone Mar 19, 2018
@manuroe
Copy link
Member Author

manuroe commented Mar 26, 2018

There is something in the app that makes peek and pork work badly but what?

The quickest fix is to disable 3D touch. It can be done by replacing https://github.com/matrix-org/matrix-ios-kit/blob/b49b3a0b6b3a6c213b29b62e09edaf0caa5792a2/MatrixKit/Views/RoomBubbleList/MXKRoomBubbleTableViewCell.m#L1189

by:

- (BOOL)textView:(UITextView *)textView shouldInteractWithURL:(nonnull NSURL *)URL inRange:(NSRange)characterRange interaction:(UITextItemInteraction)interaction
{
    BOOL shouldInteractWithURL = NO;
    if (@available(iOS 10.0, *))
    {
        if (interaction != UITextItemInteractionPreview)
        {
            shouldInteractWithURL = [self textView:textView shouldInteractWithURL:URL inRange:characterRange];
        }
    }

    return shouldInteractWithURL;
}

@manuroe
Copy link
Member Author

manuroe commented Mar 5, 2019

Fixed by #2308.

@manuroe manuroe closed this as completed Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems UX
Projects
None yet
Development

No branches or pull requests

1 participant