Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

Tap vs Click vs tappable (click) each have problems. #606

Open
ionitron-bot bot opened this issue Nov 29, 2018 · 2 comments
Open

Tap vs Click vs tappable (click) each have problems. #606

ionitron-bot bot opened this issue Nov 29, 2018 · 2 comments
Labels

Comments

@ionitron-bot
Copy link

ionitron-bot bot commented Nov 29, 2018

Original issue by @augerm on 2018-04-03T22:37:04Z

Ionic version: (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ ] 3.x
[X] 4.x

I'm submitting a ... (check one with "x")
[X] bug report
[ ] feature request

Current behavior:

  1. (click) is inconsistent on ios sometimes taking multiple times to register.
  2. (tap) fires the event on parent elements before child elements giving no chance to do an $event.stopPropagation
  3. tappable (click) causes an issue where if you scroll to short a distance (less than an inch) it causes the click event to trigger.

Appreciate any workaround or suggestions to deal with this. Thanks in advance!!

Expected behavior:

Need a way of listening to tap events across devices consistently.

Steps to reproduce:
For 2 here's a sample repo to reproduce
https://github.com/augerm/tap-ionic-bug

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):


cli packages: (D:\froxii-v3\node_modules)

    @ionic/cli-utils  : 1.19.1
    ionic (Ionic CLI) : 3.19.1

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.8
    Cordova Platforms  : android 6.3.0
    Ionic Framework    : ionic-angular 3.4.2

System:

    Android SDK Tools : 26.1.1
    Node              : v8.10.0
    npm               : 5.6.0
    OS                : Windows 10

Environment Variables:

    ANDROID_HOME : C:\Users\Michael\AppData\Local\Android\sdk

Misc:

    backend : pro
@Eptick
Copy link

Eptick commented Sep 23, 2019

Why was this moved to Ionic 3, when the package.json in the demo app is clearly defined as ionic 4.
"@ionic-native/core": "4.5.3"

@vohrahul
Copy link

Why was this moved to Ionic 3, when the package.json in the demo app is clearly defined as ionic 4.
"@ionic-native/core": "4.5.3"

It was moved automatically by the bot. If you feel this belongs to v4, please raise this to ionic framework team.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants