Navigation Menu

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: click listener taking time to respond #22177

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

bug: click listener taking time to respond #22177

indraraj26 opened this issue Sep 25, 2020 · 11 comments
Labels

Comments

@indraraj26
Copy link

indraraj26 commented Sep 25, 2020

Bug Report

Ionic version:

[ ] 4.x
[x] 5.x

Current behavior:

When i click on go to search button it is taking time to route to search-box
Repo: https://github.com/indraraj26/ionic5-starter-tabs-sidemenu/tree/click-listener
reproduce:

  1. ionic serve
  2. click on go to otp
  3. click on go to home
  4. click on go to search
  5. then back
  6. click on go to search

repeat 4, 5, 6 step

Expected behavior:

When you click on search button it should route instantly

Steps to reproduce:

Related code:
image

image

insert short code snippets here

Other information:

Ionic info:

ionic info

Ionic:

   Ionic CLI                     : 6.11.0 (C:\Users\indra26\AppData\Roaming\npm\node_modules\@ionic\cli)
   Ionic Framework               : @ionic/angular 5.3.2
   @angular-devkit/build-angular : 0.1000.8
   @angular-devkit/schematics    : 10.0.8
   @angular/cli                  : 10.0.8
   @ionic/angular-toolkit        : 2.3.3

Cordova:

   Cordova CLI       : 10.0.0
   Cordova Platforms : android 9.0.0
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.2.1, (and 6 other plugins)

Utility:

   cordova-res : 0.15.1
   native-run  : 1.0.0

System:

   Android SDK Tools : 26.1.1 (C:\Users\indra26\AppData\Local\Android\Sdk)
   NodeJS            : v10.16.3 (C:\Program Files\nodejs\node.exe)
   npm               : 6.9.0
   OS                : Windows 10

@ionitron-bot ionitron-bot bot added the triage label Sep 25, 2020
@liamdebeasi
Copy link
Member

Thanks for the issue. What device are you testing this on? I am getting a ~180ms initial load for the page, but subsequent visits to the page are fine.

@liamdebeasi liamdebeasi added the needs: reply the issue needs a response from the user label Sep 25, 2020
@ionitron-bot ionitron-bot bot removed the triage label Sep 25, 2020
@indraraj26
Copy link
Author

i have tested on chrome desktop browser as well as on device with android 7.1.2 chrome engine 85 something.
Everything is up to date.

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Sep 25, 2020
@liamdebeasi
Copy link
Member

What kind of Android device?

@liamdebeasi liamdebeasi added the needs: reply the issue needs a response from the user label Sep 25, 2020
@ionitron-bot ionitron-bot bot removed the triage label Sep 25, 2020
@indraraj26
Copy link
Author

Redmi 4A (Mi brand)
Screenshot_2020-09-26-00-34-15-471_com android settings

@ionitron-bot ionitron-bot bot added triage and removed needs: reply the issue needs a response from the user labels Sep 25, 2020
@indraraj26
Copy link
Author

indraraj26 commented Sep 28, 2020

From browser mobile mode

if you select pixel 2 device
output:
image

if you select Galaxy S5 device
output:
image

if you enable enableTracing: true app-routing.module.ts
output:
image

If you replace ion-row and ion-col with div only first navigation you might get ~180ms and subsequent visit to search-box is working fine.

====================================================

From my physical device android redmi 4a 7.12
image

VIDEO: https://www.youtube.com/watch?v=VlsbbX51-fw

@indraraj26
Copy link
Author

Hey @liamdebeasi ,

what might be the issue ?

Thank you

@indraraj26
Copy link
Author

Hi,

Ionic team any updates ? what could be the issue ?

Thank You

@liamdebeasi
Copy link
Member

Hi there,

We've been looking at this, and it looks like something in Angular is causing this issue. We are going to get in touch with an Angular team member to get a better idea of what is going on and will provide an update here when we have it.

@JosepAlacid

This comment has been minimized.

@liamdebeasi
Copy link
Member

Thanks for the issue, and apologies for the delay. After updating to the latest versions of Ionic and Angular, I am no longer able to reproduce this behavior.

I am going to close this, but if you run into this again please feel free to open another issue with a code reproduction. Thanks!

@ionitron-bot
Copy link

ionitron-bot bot commented Mar 26, 2022

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 Mar 26, 2022
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

3 participants