long pressing a link doesn't open the context menu on iOS #10622

Activity

1. Splaktar

   self-assigned this

   on Dec 18, 2018
2. Splaktar

   added

   os: iOSThis issue is specific to iOSThis issue is specific to iOS

   browser: Safari

   and removed

   browser: Safari

   on Dec 18, 2018
3. Splaktar

   added this to the 1.1.12 milestone on Dec 18, 2018
4. Splaktar

   added

   ux: integration

   type: bug

   P3: importantImportant issues that really should be fixed when possible.Important issues that really should be fixed when possible.

   needs: investigationThe cause of this issue is not well understood and needs to be investigated by the team or communityThe cause of this issue is not well understood and needs to be investigated by the team or community

   on Dec 18, 2018
5. Splaktar

   changed the title [-]iOS Safari/Chrome - Touch and Hold on the link cannot bring up context menu[/-] [+]long pressing a link doesn't open the context menu[/+] on Dec 18, 2018
6. Splaktar

   changed the title [-]long pressing a link doesn't open the context menu[/-] [+]long pressing a link doesn't open the context menu on iOS[/+] on Dec 18, 2018
7. Splaktar commented on Dec 18, 2018

   

   Splaktar

   on Dec 18, 2018

   Contributor

   More actions

   I can reproduce this with AngularJS Material 1.1.10 and HEAD (1.1.11) using iOS 12.1.1 on both Chrome and Safari using the docs. On another random web page, long pressing links does give a menu, but not on our docs.
8. Splaktar commented on Dec 18, 2018

   

   Splaktar

   on Dec 18, 2018

   Contributor

   More actions

   This appears to be related to our use of https://developer.mozilla.org/en-US/docs/Web/CSS/-webkit-touch-callout on the body. We apply -webkit-touch-callout: none;, but it's not clear why.

   I tracked this style back over 5+ years ago when the project first started and at no point could I find a reason that it was added. Based on this article it appears to be a recommended pattern for PhoneGap/Cordova/Ionic apps so that a "web" looking menu doesn't show up in your Hybrid app that is trying to fake like it's Native.

   For reference (from oldest to latest):

   • https://github.com/angular/material/blame/ac1cea4c85e393046f24017545e00207ad41d8c1/src/core/style/_structure.scss
   • https://github.com/angular/material/blame/00f4cc6d1329f618d595f887c0a13371bcff2d9e/src/core/style/structure.scss
   • https://github.com/angular/material/blame/6af1546da48aa335ca65ff32f09e2d3b69d0a2d9/src/core/style/typography.scss

   Other related issues I found:
   handsontable/handsontable#5171

   Workaround:

   body {
     -webkit-touch-callout: default;
   }
9. Splaktar

   added

   needs: feedbackThe issue creator or community need to respond to questions in this issueThe issue creator or community need to respond to questions in this issue

   and removed

   needs: investigationThe cause of this issue is not well understood and needs to be investigated by the team or communityThe cause of this issue is not well understood and needs to be investigated by the team or community

   on Dec 18, 2018
10. 20 remaining items

    Load more
11. Splaktar

    added this to the 1.2.1 milestone on May 6, 2020
12. Splaktar

    removed

    needs: feedbackThe issue creator or community need to respond to questions in this issueThe issue creator or community need to respond to questions in this issue

    on Sep 9, 2020
13. Splaktar commented on Sep 9, 2020

    

    Splaktar

    on Sep 9, 2020

    Contributor

    More actions

    There has been no feedback on this in almost 2 years, so I think that it's plenty safe to change it to default.
14. Splaktar

    added

    for: external contributor

    good first issue

    hacktoberfest

    on Sep 9, 2020
15. Splaktar

    added a commit that references this issue on Sep 21, 2020

    fix(typography): enable iOS long-press context menus

    ...

    This issue will close once commit 77e3d53 is merged into the 'master' branch.

    77e3d53
16. Splaktar

    added

    has: Pull RequestA PR has been created to address this issueA PR has been created to address this issue

    ui: CSS

    and removed

    for: external contributor

    good first issue

    hacktoberfest

    on Sep 21, 2020
17. Splaktar

    closed this as completedin #12016on Sep 21, 2020
18. Splaktar

    added a commit that references this issue on Sep 21, 2020

    fix(typography): enable iOS long-press context menus

    ...

    This issue will close once commit 3d98b6e is merged into the 'master' branch.

    3d98b6e
19. Splaktar

    added

    resolution: fixed

    on Sep 21, 2020