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.

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

amexio-layout-grid is broken in version 5.11.1 #125

Closed
baminmru opened this issue May 7, 2019 · 10 comments
Closed

amexio-layout-grid is broken in version 5.11.1 #125

baminmru opened this issue May 7, 2019 · 10 comments
Assignees

Comments

@baminmru
Copy link
Contributor

@baminmru baminmru commented May 7, 2019

I'm submitting a...


[ X] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request

Current behavior

Grid-layout do not work. See sample at
https://api.amexio.org/api/v5.11/#/layout/gridlayout-demo

Expected behavior

Grid-layout work as in prevous versions.

Minimal reproduction of the problem with instructions

For bug reports please provide the STEPS TO REPRODUCE and if possible a MINIMAL DEMO of the problem via
https://plnkr.co or similar (you can use Our Amexio Plunker as a starting point: http://plnkr.co/edit/M9AeGH?p=info)

What is the motivation / use case for changing the behavior?

Environment


Angular version: 6.2.4


Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ X] Firefox version 58
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX
 
For Tooling issues:
- Node version: 10.11
- Platform:  Windows 

Others:

@ketan-gote
Copy link
Contributor

@ketan-gote ketan-gote commented Jun 14, 2019

@baminmru - We have verified this in latest version and it is working. Can you reverify at your end and let us know. If issue is still present, please share stackblitz example, as it will help us in investigating further.
Also you can close this issue if it is resolve.

@baminmru
Copy link
Contributor Author

@baminmru baminmru commented Jun 18, 2019

Sorry, but what latest version you verify?
This feature do not work at your demo site for all 5.1x version.
Here: https://api.amexio.org/api/v5.14/#/layout/gridlayout-demo
And here: https://api.amexio.org/api/v5.13/#/layout/gridlayout-demo
And here: https://api.amexio.org/api/v5.12/#/layout/gridlayout-demo
And here: https://api.amexio.org/api/v5.11/#/layout/gridlayout-demo

'Live page' used amexio 5.4 where layout work ok.

@baminmru
Copy link
Contributor Author

@baminmru baminmru commented Jun 18, 2019

изображение

Browser version is Firefox 67.0.2

@arafkarsh
Copy link
Contributor

@arafkarsh arafkarsh commented Jun 18, 2019

It's working in Chrome and Microsoft Edge Browser and not working in FireFox. We will look into this asap.
Amexio-API-Edge

@baminmru
Copy link
Contributor Author

@baminmru baminmru commented Jun 19, 2019

I hope you fix problem for FireFox. Without this fix I cannot use new version of Amexio in my projects :-(

@ketan-gote
Copy link
Contributor

@ketan-gote ketan-gote commented Jun 19, 2019

@baminmru - We are looking into this issue and fix will be supporting FireFox also.

@arafkarsh
Copy link
Contributor

@arafkarsh arafkarsh commented Jun 20, 2019

We will be releasing the fix today.

@arafkarsh
Copy link
Contributor

@arafkarsh arafkarsh commented Jun 20, 2019

Checked the fix in the following Browsers

  • FireFox
  • Chrome
  • Safari
  • Edge
  • Opera

Amexio-fix-Grid-Layout-FireFox
Amexio-fix-Grid-Layout-Chrome
Amexio-fix-Grid-Layout-Safari
Amexio-fix-Grid-Layout-Edge

@baminmru
Copy link
Contributor Author

@baminmru baminmru commented Jun 20, 2019

Thanks, I am waiting while next release is ready.

@rashmithakkar
Copy link
Contributor

@rashmithakkar rashmithakkar commented Jun 20, 2019

Above issue is resolved in amexio version 5.14.3.
Please refer [Sample grid layout] (https://stackblitz.com/edit/amexio-v4-gridlayoutbasic-latest1?file=app/app.component.ts) with latest version.

@baminmru baminmru closed this Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants