-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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: Side Menu expose-aside-when directive doesn't work #3244
Comments
Did a bit more digging and it's not |
This bug is also in v1.0.0-rc.0, and in nightlies after the version 1.0.0-beta.14-nightly-1042. |
same over here |
+1. Just updated from beta 14 to rc.0 and now the side menu is broken because of |
+1, would love to see this feature working as planned. |
+1! |
I saw this as well in rc0. I just added the following line for now
|
+1. This line seems to be the problem, as said in this issue |
+1 |
@mixersoft's line fixes the problem I'm having. My problem: What I gathered is that every time the keyboard appears/disappears, the size of the view changes, and an evaluation is made whether the side menu should be opened or closed because of the expose-aside-when directive. You can replicate this problem by: I can't seem to find any other good way of dealing with this. I've tried doing a timeout to delay the menu toggle, and it kinda works, but the time it takes for the keyboard to close varies and depends on how much content I have on my view. Also, this isn't the ideal solution, as it would feel sluggish for the user to tap and have to wait 2 seconds or so for the side menu to open. |
Duplicate of #2328 which is now fixed and will be in 1.2 |
Type: bug
Platform: all
Version: v1.0.0-rc.0-nightly-1120
Now there is an issue when my menu is closed and I press the button it opens the side menu and immediately closes it.
The menu works if I don't include this directive...
You can see this issue in CodePen
The issue is in this function:
self.exposeAside()
as this if clause doesn't return:I thank you in advance.
The text was updated successfully, but these errors were encountered: