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.

Sign up for GitHub

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

Jump to bottom

NcNavigation - Close/Open Button #4433

Closed
eldertek opened this issue Aug 17, 2023 · 3 comments · Fixed by #4546
Closed

NcNavigation - Close/Open Button #4433

eldertek opened this issue Aug 17, 2023 · 3 comments · Fixed by #4546
Labels
bug Something isn't working feature: app-navigation Related to the app-navigation component high High priority regression Regression of a previous working feature
Milestone
8.0.0

Comments

@eldertek
Copy link

Hello, there is an issue with the close/open button since 8, I can't click on after being closed.

7.12.2 :
7 12 2

8.0.0-beta.3:
8 0 0-beta 3
@ShGKme
Copy link
Contributor

ShGKme commented Aug 18, 2023

Regression of #3114

NcAppNavigation has disabled user interaction when closed with new inert while the open button is also a part of it

@ShGKme
Copy link
Contributor

ShGKme commented Aug 18, 2023

It seems to me that it is not possible to disable inert for an element inside active inert. So we need to move the button out of navigation or use other approach rather than inert 😓

@eldertek
Copy link
Author

It seems to me that it is not possible to disable inert for an element inside active inert. So we need to move the button out of navigation or use other approach rather than inert 😓

It is crucial to have a solution to the issue :/

@raimund-schluessler raimund-schluessler added bug Something isn't working feature: app-navigation Related to the app-navigation component high High priority regression Regression of a previous working feature labels Aug 25, 2023
@raimund-schluessler raimund-schluessler added this to the 8.0.0 milestone Aug 25, 2023
@raimund-schluessler raimund-schluessler mentioned this issue Aug 25, 2023
Merged
1 task
@susnux susnux mentioned this issue Sep 19, 2023
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working feature: app-navigation Related to the app-navigation component high High priority regression Regression of a previous working feature
Projects
None yet
Milestone
8.0.0
3 participants
@raimund-schluessler @ShGKme @eldertek