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

[IONIC 4] Ignored attempt to cancel a touchmove event with cancelable=false, #15256

Closed
wenisy opened this issue Aug 21, 2018 · 36 comments
Closed
Labels
package: core @ionic/core package

Comments

@wenisy
Copy link

wenisy commented Aug 21, 2018

Bug Report

Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.

   ionic (Ionic CLI)          : 4.1.0
   Ionic Framework            : @ionic/angular 4.0.0-beta.3
   @angular-devkit/core       : 0.7.2
   @angular-devkit/schematics : 0.7.2
   @angular/cli               : 6.1.2
   @ionic/ng-toolkit          : 1.0.5
   @ionic/schematics-angular  : 1.0.4

Describe the Bug

I add a tap event on ion-item, and to make it could scroll I also add [style.touch-action]="'pan-y'" to make it work.

Pull down to refresh the page, there is a error in console.
[Intervention] Ignored attempt to cancel a touchmove event with cancelable=false, for example because scrolling is in progress and cannot be interrupted.

Steps to Reproduce
Steps to reproduce the behavior:

  1. Pull down the page
  2. Check console, you could get the error.

Related Code

<ion-content>
    <ion-refresher>
    </ion-refresher>
    <ion-list>
        <ion-item (tap)="xxxx()" `[style.touch-action]="'pan-y'">
    </ion-list>
</ion-content>

Expected Behavior
No error in console.

Additional Context
image

@ionitron-bot ionitron-bot bot added the triage label Aug 21, 2018
@akkitheakhil
Copy link

I have the same issue. Also getting an error "refresher.complete is not a function"
any solution to this ?

@wenisy
Copy link
Author

wenisy commented Aug 31, 2018

@akkitheakhil
still waiting...

If you get any solution of this, could you please write it down here?

@RHinderiks
Copy link

Same here

@copaste
Copy link

copaste commented Sep 6, 2018

I have the same issue.

@Silvest89
Copy link

Silvest89 commented Sep 12, 2018

Having it here as well

ionic (Ionic CLI) : 4.1.1 (/usr/local/lib/node_modules/ionic)
Ionic Framework : @ionic/angular 4.0.0-beta.8
@angular-devkit/core : 0.8.1
@angular-devkit/schematics : 0.8.1
@angular/cli : 6.2.1
@ionic/ng-toolkit : 1.0.8
@ionic/schematics-angular : 1.0.6

@babaru
Copy link

babaru commented Sep 19, 2018

Guys, please check Ionic 4 beta docs: https://beta.ionicframework.com/docs/api/refresher

@wenisy
Copy link
Author

wenisy commented Sep 26, 2018

Guys, please check Ionic 4 beta docs: https://beta.ionicframework.com/docs/api/refresher
@babaru

Sorry, I didn't get your point.

You mean we should add

setTimeout(() => {
      console.log('Async operation has ended');
      event.target.complete();
    }, 2000);

In doRefresh function?

I have done that, but still have this issue.

@natalieG96
Copy link

+1

@plebani
Copy link

plebani commented Oct 15, 2018

I have same problem.
image

Code:

image

@petinho
Copy link

petinho commented Oct 20, 2018

Same problem here!

@kelvinjp
Copy link

+1

@petinho
Copy link

petinho commented Oct 21, 2018

It's solved for me. It was my fault. Now I have the element empty. The component is working and there's no error on the console.
My code:

<ion-content padding>
  <ion-refresher  (ionRefresh)="doRefresh($event)">
        <ion-refresher-content>          
        </ion-refresher-content>
  </ion-refresher>
  <ion-card *ngFor="let ticket of currentTickets">
    <ion-card-header>
      {{ticket.Title}}
    </ion-card-header>
    <ion-card-content>
      <div>{{ticket.Description}}</div>
    </ion-card-content>
  </ion-card>
</ion-content>

@Jerga99
Copy link

Jerga99 commented Oct 31, 2018

I am still having this issue. Is there any known fix ?

@nbadiganti
Copy link

nbadiganti commented Nov 2, 2018

Still issue persists in the latest ionic version. Any known fix or turn around solution?

@Riyaz0001
Copy link

same problem here.
ionic Info:
`Ionic:

ionic (Ionic CLI) : 4.3.1 (C:\Users\Riyaz\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework : @ionic/angular 4.0.0-beta.15
@angular-devkit/build-angular : 0.8.7
@angular-devkit/schematics : 0.8.7
@angular/cli : 6.2.7
@ionic/angular-toolkit : 1.1.0

Cordova:

cordova (Cordova CLI) : 8.1.2 (cordova-lib@8.1.1)
Cordova Platforms : android 7.1.1
Cordova Plugins : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.2.0, (and 11 other plugins)

System:

NodeJS : v8.11.2 (C:\Program Files\nodejs\node.exe)
npm : 5.6.0
OS : Windows 10`

@sashayello
Copy link

Same problem. Have some idea to fix this?

@sneko
Copy link

sneko commented Nov 23, 2018

Up. Same error for me

@ashvinmay
Copy link

same problem any idea to fix

@XieGuoJuan
Copy link

Is there a solution?

@paulstelzer paulstelzer added needs: investigation package: core @ionic/core package and removed triage labels Dec 3, 2018
@paulstelzer
Copy link
Contributor

This is still not working with beta.17?

@KevinKelchen
Copy link

I'm not the OP, but FWIW, I still see the error with Beta 17 in our application.

@paulstelzer
Copy link
Contributor

paulstelzer commented Dec 5, 2018

Thanks for confirming it :)

@kharatps02
Copy link

same issue for us. waiting for fix.

@paulstelzer
Copy link
Contributor

It would be great if someone can create a repo for that so we can see the console error log :)

@LautaroLorenz
Copy link

same issue

@KevinKelchen
Copy link

Here you go, @paulstelzer.

Steps to Reproduce

  1. Clone this repo on your machine: https://github.com/KevinKelchen/ionic4-refresher-error . It is a very slightly modified version of the blank Angular starter template. The only substantial change is adding the ion-refresher. It is on Beta 19.
  2. In the terminal, path into the repo root and run npm i.
  3. Run ionic serve.
  4. Open the Google Chrome Dev Tools so you can see both the Dev Tools Console as well as the running application.
  5. Pull down on the text content ("The world is your oyster. ...") and it will reveal the refresher.
    • Note: You will need to reload the browser window after the refresh loading spinner appears if you want to try the refresher again. This is a pretty simple sample.
  6. If you pull down at a slower/moderate rate, you will see one or more instances of the error in the console:
    image

@avuenja
Copy link

avuenja commented Dec 21, 2018

same issue here

edit
I don't know if it helps you, but, I fixed the error in my code, adding async to the called functions in my doRefresh and removed the reset of the variables.

@sivadinesh1
Copy link

Had same error. fixed the error by removing async in doRefresh method. there was two async in doRefresh and in calling http client api

@xiaoxiao95888
Copy link

same error

@BassamHabash
Copy link

Hi All, For resolving the complete in not a function, add a variable reference to html
<ion-refresher #refresherRef slot="fixed" (ionRefresh)="doRefresh($event)">

and in .ts add :
@ViewChild('refresherRef') refresherRef;

doRefresh(refresher) {
    console.log('Begin async operation', refresher);

    setTimeout(() => {
      console.log('Async operation has ended');
      this.refresherRef.complete();
    }, 2000);
  }

and for that error, I have add content empty:

 <ion-refresher-content>
    </ion-refresher-content>

and that work for me.

@3adeling
Copy link

3adeling commented Jan 11, 2019

I am getting the same error and it is causing me another problem. check my post:

https://forum.ionicframework.com/t/video-refresher-gets-triggered-when-scrolling-down/152807

for my case, I noticed that when I pull the refresher gently and slowly the problem is gone and scroll works fine.

to generate the scrolling issue:

  • pull the refresher and let it go up by itself.
  • scrolling down won't work, instead it will call the doRefresh function.

to avoid the scrolling issue:

  • pull the refresher gently and slowly until the 'up' arrow appears
  • release it.
  • scrolling works fine

@mtshare
Copy link

mtshare commented Feb 12, 2019

Same here +1

@sivadinesh1
Copy link

Same error in latest build too +1

@liamdebeasi
Copy link
Contributor

Hi everyone,

This has been resolved via #17351 and will be in the next release of Ionic. I have published a nightly build with this fix if anyone would be interested in testing with their apps (4.1.0-dev.201903011951.f205b10). Thanks!

michaeltintiuc pushed a commit to ModusCreateOrg/ionic that referenced this issue Mar 4, 2019
…-team#17351)

fixes ionic-team#15256

Add loading controller vue docs

Add ion-tabs support, QOL fixes

Fix @ionic/core version, rebuild core to include docs

Update router
@rafalschmidt97
Copy link

rafalschmidt97 commented Mar 10, 2019

Issue still exists. I get this only on android (for example while moving slider). Never on ios.

Screenshot 2019-03-10 at 09 41 55

Ionic:

   ionic (Ionic CLI)             : 4.9.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.1.1
   @angular-devkit/build-angular : 0.12.4
   @angular-devkit/schematics    : 7.2.4
   @angular/cli                  : 7.2.4
   @ionic/angular-toolkit        : 1.2.3

Cordova:

   cordova (Cordova CLI) : 8.1.2 (cordova-lib@8.1.1)
   Cordova Platforms     : android 8.0.0, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.3.2, (and 10 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/rafalschmidt/Library/Android/sdk)
   ios-deploy        : 1.9.4
   ios-sim           : 7.0.0
   NodeJS            : v10.15.0 (/usr/local/bin/node)
   npm               : 6.4.1
   OS                : macOS Mojave
   Xcode             : Xcode 10.1 Build version 10B61

@liamdebeasi
Copy link
Contributor

Hi there,

This issue is related to the touchmove event on the refresher component. For anything outside of that, please create a new issue.

Thanks!

@ionic-team ionic-team locked and limited conversation to collaborators Mar 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: core @ionic/core package
Projects
None yet
Development

No branches or pull requests