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

I want the dismiss animation in front of the menu,but set dismissOnPresent = false and presentationStyle = menuSlideIn doesn't work #671

Open
xiaoyao20084321 opened this issue Apr 16, 2021 · 1 comment

Comments

@xiaoyao20084321
Copy link

xiaoyao20084321 commented Apr 16, 2021

I have read the guidelines for contributing and I understand

  • [✔️] My issue is happening in the latest version of SideMenu (older versions are no longer maintained).
  • [✔️] My issue was not solved in the README.
  • [✔️] My issue can not be answered on stackoverflow.com.
  • [✔️] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
  • [✔️] My issue is reproducible in the demo project.

Describe the bug
When I set dismissOnPresent = false and presentationStyle = menuSlideIn, when the VC dismiss from the menu present,the dismiss animation from the back of the menu will appear, instead of the dismiss animation in front of the menu,how to fix this bug?

To Reproduce
Steps to reproduce the behavior:

  1. write dismissOnPresent = false
  2. write presentationStyle = .menuSlideIn
  3. Click on Demo right menu
  4. Click on Present View Controller 1
  5. Click on dismiss

Expected behavior
I want the dismiss animation in front of the menu

Screenshots
image
image

Additional context
Add any other context about the problem here.

@nicksterious
Copy link

Were you ever able to sort this out? Seem the MenuSettings don't work at all or perhaps there's a counterintuitive way to apply them?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants