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: refresher gesture always takes priority over menu gesture #27266

Open
3 tasks done
dchase-sir opened this issue Apr 24, 2023 · 2 comments
Open
3 tasks done

bug: refresher gesture always takes priority over menu gesture #27266

dchase-sir opened this issue Apr 24, 2023 · 2 comments
Labels
package: core @ionic/core package type: bug a confirmed bug report

Comments

@dchase-sir
Copy link

dchase-sir commented Apr 24, 2023

Prerequisites

Ionic Framework Version

v6.x

Current Behavior

Angular, when an ion-refresher is placed onto a page, the ionic sidemenu (splitpane/menu/router) will not react to dragging from the left in the Chrome emulator. I don't know if it does this on a device or not.

Expected Behavior

The side-menu would continue working with an ion-refresher on the page.

Steps to Reproduce

  1. Create a new sidemenu project using the Ionic/Angular.
  2. Add ion-refresher to folder.page.html.
  3. Run the project with ionic serve.
  4. Change to Mobile Emulation in the browser.
  5. Attempt to drag from the left, to slide the menu out.
  6. If that just worked, refresh, it doesn't work most of the time.
  7. Try dragging at the top of the window, in the header instead of the content, and it works every time.
  8. Try pulling to refresh, during the refresh, the menu works normally until the refresh window collapses.
  9. After testing, remove the ion-refresher and it works every time.

Below is my refresher code:

<ion-refresher slot="fixed" (ionRefresh)="handleRefresh($event)">
      <ion-refresher-content
          pullingIcon="chevron-down-circle-outline"
          pullingText="Pull to refresh"
          refreshingSpinner="circles"
          refreshingText="Refreshing...">
      </ion-refresher-content>
</ion-refresher>

handleRefresh(ev : any) { 
};

When the refresh window is active, the sidemenu work again for a second, but as soon the refresh content collapses, it no longer works again.
The sidemenu still works if try to drag on the heading of the page (ion-header) instead of the content.
While testing with a new project, I found that sometimes it does work, then refresh and it no longer works.
I've tried adding padding, margins, and additional div's.

Code Reproduction URL

https://github.com/dchase-sir/slidertest.git

Ionic Info

Ionic:

Ionic CLI : 6.16.3 (C:\Users\jhsdjh\AppData\Roaming\npm\node_modules@ionic\cli)
Ionic Framework : not installed
@angular-devkit/build-angular : 15.2.6
System:

NodeJS : v14.20.1 (C:\Program Files\nodejs\node.exe)
npm : 6.14.17
OS : Windows 10

Additional Information

If I take the ion-refresher off the page it works every time. Every once in a while I'm tricked into thinking I figured it out, when it works ONCE then I refresh the browser and never again. I can still trigger the menu using an icon in the header, so I know the sidemenu still works, I think something is just blocking the target when the two are used together.

@ionitron-bot ionitron-bot bot added the triage label Apr 24, 2023
@dchase-sir dchase-sir changed the title bug: bug: Slider does not respond to gesture after adding ion-refresher to the page, unless in mid-refresh Apr 24, 2023
@dchase-sir dchase-sir changed the title bug: Slider does not respond to gesture after adding ion-refresher to the page, unless in mid-refresh bug: Slider does not respond to dragging from the left after adding ion-refresher to the page, unless in mid-refresh Apr 24, 2023
@liamdebeasi liamdebeasi added the ionitron: needs reproduction a code reproduction is needed from the issue author label Apr 24, 2023
@ionitron-bot
Copy link

ionitron-bot bot commented Apr 24, 2023

Thanks for the issue! This issue has been labeled as needs reproduction. This label is added to issues that need a code reproduction.

Please reproduce this issue in an Ionic starter application and provide a way for us to access it (GitHub repo, StackBlitz, etc). Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.

If you have already provided a code snippet and are seeing this message, it is likely that the code snippet was not enough for our team to reproduce the issue.

For a guide on how to create a good reproduction, see our Contributing Guide.

@ionitron-bot ionitron-bot bot removed the triage label Apr 24, 2023
@dchase-sir
Copy link
Author

I have added this to a public repository https://github.com/dchase-sir/slidertest.git

@amandaejohnston amandaejohnston added triage and removed ionitron: needs reproduction a code reproduction is needed from the issue author labels Apr 25, 2023
@liamdebeasi liamdebeasi changed the title bug: Slider does not respond to dragging from the left after adding ion-refresher to the page, unless in mid-refresh bug: refresher gesture always takes priority over menu gesture Jul 6, 2023
@liamdebeasi liamdebeasi added package: core @ionic/core package type: bug a confirmed bug report labels Jul 6, 2023
@ionitron-bot ionitron-bot bot removed triage labels Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

No branches or pull requests

3 participants