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

bug: status bar scroll to top does not fully cancel content scroll #20423

Closed
EinfachHans opened this issue Feb 10, 2020 · 9 comments · Fixed by #24001
Closed

bug: status bar scroll to top does not fully cancel content scroll #20423

EinfachHans opened this issue Feb 10, 2020 · 9 comments · Fixed by #24001
Labels
package: core @ionic/core package type: bug a confirmed bug report

Comments

@EinfachHans
Copy link
Contributor

EinfachHans commented Feb 10, 2020

Bug Report

Ionic version:

[x] 4.11.10
maybe also a topic for v5?

Current behavior:

When i have a large content on iOS and scroll down, then (while scrolling) tap on the status bar, a scroll to top is emitted. So far so good. But after that, the content is automatically scrolled a bit, like it scrolls the rest of the first scroll.

Expected behavior:

When i tap on the status bar and scroll to top, want to end the current scroll and don't continue scrolling.

Video

Video-100220-09-35-26

Other information:

Ionic info:

Ionic:

   Ionic CLI                     : 6.0.1 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 4.11.10
   @angular-devkit/build-angular : 0.803.25
   @angular-devkit/schematics    : 8.3.25
   @angular/cli                  : 8.3.25
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 9.0.0 ([email protected])
   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 52 other plugins)

Utility:

   cordova-res (update available: 0.9.0) : 0.8.1
   native-run                            : 0.3.0

System:

   Android SDK Tools : 25.2.3 (/Users/hanskrywalsky/Library/Android/sdk)
   ios-deploy        : 1.10.0
   ios-sim           : 8.0.2
   NodeJS            : v10.16.3 (/usr/local/bin/node)
   npm               : 6.13.7
   OS                : macOS Catalina
   Xcode             : Xcode 11.3.1 Build version 11C504
@EinfachHans
Copy link
Contributor Author

EinfachHans commented Feb 29, 2020

I did some Research about this Topic interrupt Momentum Scroll and found this:

The trick is to set overflow: hidden, which stops the inertia/momentum scrolling.

@liamdebeasi
Copy link
Contributor

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue?

@liamdebeasi liamdebeasi added needs: reply the issue needs a response from the user and removed needs: investigation labels Mar 2, 2020
@EinfachHans
Copy link
Contributor Author

EinfachHans commented Mar 2, 2020

Yeah, i can do it, but not before tomorrow. But you can just create a new App with a large List and run on an iOS Device.

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Mar 2, 2020
@liamdebeasi
Copy link
Contributor

Thanks for the follow up. Are you using Cordova or Capacitor?

@liamdebeasi liamdebeasi added the needs: reply the issue needs a response from the user label Mar 2, 2020
@ionitron-bot ionitron-bot bot removed the triage label Mar 2, 2020
@EinfachHans
Copy link
Contributor Author

Cordova, because not all of my Plugins are Capacitor-Ready

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Mar 2, 2020
@liamdebeasi
Copy link
Contributor

Ok thanks I was able to reproduce this. There are some issues with the status bar tap in both Cordova and Capacitor so I will need to dig into this a bit more.

@EinfachHans
Copy link
Contributor Author

@liamdebeasi any chance you can dig into this a bit more for V6?

@liamdebeasi liamdebeasi changed the title bug: Status Bar Scroll To Top doesn't end current Scroll bug: status bar scroll to top does not fully cancel content scroll Sep 16, 2021
@liamdebeasi liamdebeasi added package: core @ionic/core package type: bug a confirmed bug report and removed needs: investigation labels Sep 16, 2021
@liamdebeasi liamdebeasi added type: bug a confirmed bug report and removed type: bug a confirmed bug report labels Sep 30, 2021
@liamdebeasi
Copy link
Contributor

Thanks for the issue. This has been resolved via #24001, and a fix will be available in an upcoming release of Ionic Framework.

@ionitron-bot
Copy link

ionitron-bot bot commented Oct 30, 2021

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Oct 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants