aurelia-ui-toolkits / aurelia-materialize-bridge

Materialize CSS components for Aurelia
http://aurelia-ui-toolkits.github.io/demo-materialize/
MIT License
155 stars 53 forks source link

Select, Edge, Gist and the newest Aurelia #528

Closed nndevstudio closed 6 years ago

nndevstudio commented 6 years ago

We are facing very strange behaviour of select component with the newest Aurelia components in the Edge browser. Only the first selection of any element in list is ok, any next selection without focus change causes page reload. So I want to prepare a Gist example: https://gist.run/?id=79bd16bb78646370ee70b93e97287e88

Where can we find the newest bundle of Aurelia? I want to change at least this line: <script src="https://rawgit.com/aurelia-ui-toolkits/aurelia-materialize-bundles/0.30.0/config2.js"></script> and try to cause the same behaviour.

MaximBalaganskiy commented 6 years ago

Run any example from the demo app, it will generate a gist with latest bundles

On Sat., 1 Sep. 2018, 1:31 am Nemanja Novicic, notifications@github.com wrote:

We are facing very strange behaviour of select component with the newest Aurelia components in the Edge browser. Only the first selection of any element in list is ok, any next selection without focus change causes page reload. So I want to prepare a Gist example: https://gist.run/?id=5b2726618189067957bfda2901fa8b65

Where can we find the newest bundle of Aurelia? I want to change at least this line: and try to cause the same behaviour.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/issues/528, or mute the thread https://github.com/notifications/unsubscribe-auth/ADimvO7JUrjEbA3fInvklubPmPPJl3aSks5uWVbTgaJpZM4WVbTF .

MaximBalaganskiy commented 6 years ago

I cannot reproduce this in Edge with the demo app

nndevstudio commented 6 years ago

For some reason, in the last few days, whenever (in any browser) I click on PLAY button (on this page, for example: https://aurelia-ui-toolkits.github.io/demo-materialize/#/samples/select/binding-to-objects) I get this redirection error: http://aurelia-ui-toolkits.github.io/demo-materialize/?error=redirect_uri_mismatch&error_description=The+redirect_uri+MUST+match+the+registered+callback+URL+for+this+application.&error_uri=https%3A%2F%2Fdeveloper.github.com%2Fapps%2Fmanaging-oauth-apps%2Ftroubleshooting-authorization-request-errors%2F%23redirect-uri-mismatch.

I am signed in on Github but no Gist page opens.

Btw, Select demo is working fine in Edge but when we copy the same code and do jspm update in our project then we face mentioned problem. We will investigate more what is happening, I've just wanted to check is Gist updated.

MaximBalaganskiy commented 6 years ago

There is an issue at the moment with my Azure Function which generates auth tokens for gists. I'll look into this tomorrow.

MaximBalaganskiy commented 6 years ago

The function is fixed now. As for not working link, please use http instead of https - this is how GitHub OAuth app is configured E.g. http://aurelia-ui-toolkits.github.io/demo-materialize/#/samples/select/binding-to-objects

nndevstudio commented 6 years ago

I have updated our example https://gist.run/?id=79bd16bb78646370ee70b93e97287e88. It is derived from Aurelia demo app now. In Gist version we are facing <select> component blocking and automatic page reload issue when try to change selected list element without focus change. Sometimes we can change max. 2-3 times selected item and than it blocks but usually only the first selection works. It is tested in 3 Edge instances. No console errors.

It is interesting that in demo app it works ok.

MaximBalaganskiy commented 6 years ago

Can you please create a simple jspm (no bundling) reproduction repository? Something is not right with Edge+Jspm...

nndevstudio commented 6 years ago

Can't make it today but I will try tomorrow.

nndevstudio commented 6 years ago

I hope this will be helpful: https://github.com/nndevstudio/aurelia-select-only-jspm-ts

MaximBalaganskiy commented 6 years ago

Not sure why this fixes the issue for Edge - probably some weird timing. But anyway...