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 [ionic5]: click twice on ion-item will dismiss swipeable modal #20608

Closed
indraraj26 opened this issue Feb 25, 2020 · 11 comments
Closed

bug [ionic5]: click twice on ion-item will dismiss swipeable modal #20608

indraraj26 opened this issue Feb 25, 2020 · 11 comments
Labels
package: core @ionic/core package type: bug a confirmed bug report
Milestone

Comments

@indraraj26
Copy link

Bug Report

Ionic version:

[x] 5.x

Current behavior:

Click two times little bit fast on ion-item it will close the swipeable modal.
Tested on: Iphone 6 iOS: 12.4.4

Expected behavior:

It should not close the modal.

Steps to reproduce:

Related code:

Repo: https://github.com/indraraj26/ionic4-starter-tabs-sidemenu/tree/ionic-5
Dashboard: Click on first ion-item to open swipeable modal.

insert short code snippets here

Other information:

Ionic info:

ionic info

Ionic:

   Ionic CLI                     : 5.2.3 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 5.0.1
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.0.0

Cordova:

   Cordova CLI       : 8.1.2 (cordova-lib@8.1.1)
   Cordova Platforms : ios 4.5.5
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 4 other plugins)

Utility:

   cordova-res : not installed
   native-run  : 0.2.8 (update available: 0.3.0)

System:

   ios-deploy : 1.9.4
   NodeJS     : v12.13.0 (/usr/local/bin/node)
   npm        : 6.12.0
   OS         : macOS High Sierra
   Xcode      : Xcode 10.1 Build version 10B61


@ionitron-bot ionitron-bot bot added the triage label Feb 25, 2020
@indraraj26 indraraj26 changed the title bug [ionic5]: click twice on ion-item swipeable modal bug [ionic5]: click twice on ion-item will dismiss swipeable modal Feb 25, 2020
@indraraj26
Copy link
Author

Hi @liamdebeasi , Can you please look into this. This is really strange/weird behaviour.

Thank you

@liamdebeasi liamdebeasi added package: core @ionic/core package type: bug a confirmed bug report labels Feb 26, 2020
@liamdebeasi liamdebeasi added this to the 5.0.3 milestone Feb 26, 2020
@ionitron-bot ionitron-bot bot removed the triage label Feb 26, 2020
@liamdebeasi
Copy link
Contributor

Thanks for the issue. Can you try the following dev build and let me know if it resolves the issue?

npm i @ionic/angular@5.1.0-dev.202003031524.f64b142

@indraraj26
Copy link
Author

Hi @liamdebeasi , Today i left from office will definitely test tomorrow. Also iOS device is in office.

Thank you for your instant solution.

@tetkosimi
Copy link
Contributor

Looks good @liamdebeasi !
Thanks!

@indraraj26
Copy link
Author

Hi @liamdebeasi, This is working great but animation is not working properly now.

Tested on: Iphone 6 iOS: 12.4.4

ISSUE:

  1. Parent page -> it doesnt have smooth animation for border-radius.
  2. Sometime modal will open without any animation.
  3. Sometime modal will overlap parent page.

Reproduce Video: https://youtu.be/RMYudiq9Dlc?t=20

Anyway for this i will create another issue template.

@liamdebeasi
Copy link
Contributor

@indraraj26 Is the issue you are reporting only happening on the dev build provided, or is it happening on the latest stable version of Ionic too?

@indraraj26
Copy link
Author

Hi @liamdebeasi ,

I have tested with your dev build what you have provided as you can see in ionic info for ionic framework version
Link: #20687

Thanks for the issue. Can you try the following dev build and let me know if it resolves the issue?

npm i @ionic/angular@5.1.0-dev.202003031524.f64b142

@liamdebeasi
Copy link
Contributor

liamdebeasi commented Mar 5, 2020

@indraraj26 I understand that, but what I am trying to determine is whether or not the issue you mentioned in #20608 (comment) occurs in the latest stable version of Ionic Framework. If it occurs in the latest stable version, then that means it is not directly related to the fix in the dev build.

The reason I ask is I would like to get this fix merged into the framework code base.

@indraraj26
Copy link
Author

Okay i will wait for merge and will directly test on latest ionic version.

@liamdebeasi
Copy link
Contributor

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

@ionitron-bot
Copy link

ionitron-bot bot commented Apr 5, 2020

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 Apr 5, 2020
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

No branches or pull requests

3 participants