-
Notifications
You must be signed in to change notification settings - Fork 130
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
Does't work in stock Andoid < 4.4 Browser ( AppleWebKit/534.30 ) #44
Comments
I have the same Issue! The rotating animation of the menue button works, but the other buttons won't appear. With Android 4.1 Cordova. I am using Ionic and I put the mfb nav in the content area: <ion-content>
<ion-list >
<ion-item ng-repeat="item in items" >
</ion-item>
</ion-list>
<nav mfb-menu position="br" effect="zoomin" label="Schließen"
active-icon="ion-close" resting-icon="ion-navicon-round"
toggling-method="click" >
<button mfb-button icon="ion-trash-b" label="Löschen" ng-click="data.showDelete = !data.showDelete;" ></button>
<button mfb-button icon="ion-document-text" ng-click="createText()" label="Text"></button>
</nav>
</ion-content> It works perfect for the Desktop browsers. But not in the cordova webview! I can see the rotation animation of the menue button. But the other buttons will be not displayed. I think it is a rendering problem ! I suggest to use Hardware-Accelerated by using transform 3d. Perhaps i find a way to modify! |
I ve tested a while. I thought css support would be the problem. But I found out that only the Android onboard browser has the problem(its also used in cordova!). With Chrome and its works perfect. In firefox there is strange behavior. I can click it once and the menue opens. But if I click again it not really close it. Only if I change the orientation of the device. The close/open sequence will triggert! And I can see the Animation Strange! |
It seems to be a repaint issue. I'll do more tests next week. |
Can't test it right now, but good to know. Keep us posted about your findings. Thanks! |
We have a solution :
allows repaint to be done. In pure CSS, without any JS. Credits : http://stackoverflow.com/a/20982320 |
Great! Will you make a PR for this? |
I don't have much time right now. I'll make a PR next week or, if you want, you may do it. |
Is it necessary to attach Ideally the directive should not affect parts of the DOM that it does not create/own. |
Hi,
In stock Android browser ( <= 4.3 ) the animation doesn't appear and menu doesn't "open".
Best regards.
The text was updated successfully, but these errors were encountered: