ionic-team / ionic-v3

The repo for Ionic 3.x. For the latest version of Ionic, please see https://github.com/ionic-team/ionic
Other
128 stars 85 forks source link

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

Open ionitron-bot[bot] opened 5 years ago

ionitron-bot[bot] commented 5 years ago

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 commented 4 years ago

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 commented 4 years ago

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.