-
Notifications
You must be signed in to change notification settings - Fork 3
Description
Slack thread: https://polymer.slack.com/archives/C04EYH0AD/p1569343016005500?thread_ts=1568378635.002200&cid=C04EYH0AD
jouni:vaadin: 24 hours ago
😅 definitely not supposed to hide itself like that! I’ve only been testing it on my iPhone 😊
leho 22 hours ago
So webkit vs gecko thing huh
leho 22 hours ago
Are you interested in taking a look at the code if I file an issue, or should I just Upwork it
jouni:vaadin: 19 hours ago
I already took a look. The problem is somehow with touch events, that in Firefox an element messes those up, and a touch event listener doesn’t get fired properly. Adding the following in j-app-layout.js fixes it:
jouni:vaadin: 19 hours ago
One addition: there’s still the problem in Firefox that you can’t scroll the menu in the drawer using touch. Didn’t have time to debug that yet.
jouni:vaadin: 19 hours ago
Re: the touch event and : I’m not sure that was actually the right fix. Doesn’t work anymore 😄