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] Closing the menu from colapsed Navbar seems laggy #2

Closed
iamf3i opened this issue Jun 26, 2019 · 3 comments
Closed

[Bug] Closing the menu from colapsed Navbar seems laggy #2

iamf3i opened this issue Jun 26, 2019 · 3 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@iamf3i
Copy link

iamf3i commented Jun 26, 2019

Version

1.0.0

Reproduction link

https://demos.creative-tim.com/argon-design-system-react/#/

Steps to reproduce

  1. Get the collapsed Navbar by reducing the window width
  2. Open the menu Toggler
  3. Close it.

The behaviour is different from other Ui kit (BLK, non react Argon, Angular Argon....)

What is expected?

Instant closing of the menu

What is actually happening?

Delay, seems laggy.


Solution

Additional comments

I'm kinda newbie but is it a css issue ? Transition ? What should I try to fix it ?

@einazare
Copy link
Contributor

Hello there, @iamf3i ,

Thank you for your interest in working with our products and for using our issue template.
At the moment we do not have a fix for this issue.
We'll continue to search for one.
Meantime, if you have a solution, please share it with us.

Best,
Manu

@einazare einazare added bug Something isn't working enhancement New feature or request labels Jun 26, 2019
@iamf3i
Copy link
Author

iamf3i commented Jul 1, 2019

The issue is related to the uncontrolled collapse component from reactstrap.
It automatically adds the "show" css class which is used to describe the animation for opening.

BUT

It doesn't add automatically the "collapsing-out" for the closing animation.

Implementing the Collapse component instead of Uncontrolled Collapse will allow us to describe the actions for the onExiting event, and put the "collapsing-out" animation to the component.

Is it worth a pull request ?

Rgds,

Fawzi

@einazare
Copy link
Contributor

einazare commented Jul 1, 2019

Hello there, @iamf3i ,

You can open a PR, but it would be great if you would open it on another branch, not on the master one, but on the v1.1.0 branch.

Best,
Manu

einazare added a commit that referenced this issue Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants