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

Problem in the view option of the toolbar (Bootstrap UI) #879

Closed
yannniskargakis opened this Issue Aug 21, 2017 · 10 comments

Comments

Projects
None yet
2 participants
@yannniskargakis

yannniskargakis commented Aug 21, 2017

It seems that there is a bug in the view functionality of the toolbar.
You cannot navigate to the previous object of the table through the arrows by selecting an object from the grid to view. Instead console returns the following error: "Uncaught ReferenceError: npos is not defined"

You can reproduce this bug in your own bootstrap examples: Fuctionality -> Toolbar Settings

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Sep 7, 2017

Owner

Please let us know if the fix is ok for you.

Owner

tonytomov commented Sep 7, 2017

Please let us know if the fix is ok for you.

@yannniskargakis

This comment has been minimized.

Show comment
Hide comment
@yannniskargakis

yannniskargakis Sep 8, 2017

Previous navigator button now works, but for some reason #779 is now back.

yannniskargakis commented Sep 8, 2017

Previous navigator button now works, but for some reason #779 is now back.

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Sep 11, 2017

Owner

Hello,

Regarding #779 - we an not reproduce the problem. Can you please send us a example with the new build where this happen?

Thank you

Owner

tonytomov commented Sep 11, 2017

Hello,

Regarding #779 - we an not reproduce the problem. Can you please send us a example with the new build where this happen?

Thank you

@yannniskargakis

This comment has been minimized.

Show comment
Hide comment
@yannniskargakis

yannniskargakis Sep 11, 2017

Hello,
It is exactly the same problem (Bootsrtap UI / Windows 7 / both Chrome 60 and Firefox 54) as before your fix on 3 Aug (see related comments on #779 ). I have reproduced the problem using my old example on #779 using your latest build.

yannniskargakis commented Sep 11, 2017

Hello,
It is exactly the same problem (Bootsrtap UI / Windows 7 / both Chrome 60 and Firefox 54) as before your fix on 3 Aug (see related comments on #779 ). I have reproduced the problem using my old example on #779 using your latest build.

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Sep 11, 2017

Owner

Sorry, I should test your example before to ask. The problem appear to be something like Catch 22 - in your example if you set shrinkToFit : true it will work with the last code correct, but if shrinkToFit false it will work with the old code (before my fix).

Will see how to solve the problem.

Regards
Tony

Owner

tonytomov commented Sep 11, 2017

Sorry, I should test your example before to ask. The problem appear to be something like Catch 22 - in your example if you set shrinkToFit : true it will work with the last code correct, but if shrinkToFit false it will work with the old code (before my fix).

Will see how to solve the problem.

Regards
Tony

@yannniskargakis

This comment has been minimized.

Show comment
Hide comment
@yannniskargakis

yannniskargakis Sep 11, 2017

Unfortunately shrinkToFit : true it's not an option for us, since we have grids with a lot of columns.
Please inform me for a fix asap.
Thanks

yannniskargakis commented Sep 11, 2017

Unfortunately shrinkToFit : true it's not an option for us, since we have grids with a lot of columns.
Please inform me for a fix asap.
Thanks

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Sep 11, 2017

Owner

Yes, I know. The script should work in both cases - I do not mean "use" shrinkToFit : true when I posted this.

Owner

tonytomov commented Sep 11, 2017

Yes, I know. The script should work in both cases - I do not mean "use" shrinkToFit : true when I posted this.

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Sep 11, 2017

Owner

Can you please check the new fix. I hope we have fixed the issue this time. Here is the commit c40aeac

Owner

tonytomov commented Sep 11, 2017

Can you please check the new fix. I hope we have fixed the issue this time. Here is the commit c40aeac

@yannniskargakis

This comment has been minimized.

Show comment
Hide comment
@yannniskargakis

yannniskargakis Sep 12, 2017

It seems that now it's ok. I hope to not bother you again with this issue :-)
Thank you for the fix

yannniskargakis commented Sep 12, 2017

It seems that now it's ok. I hope to not bother you again with this issue :-)
Thank you for the fix

@tonytomov

This comment has been minimized.

Show comment
Hide comment
@tonytomov

tonytomov Dec 13, 2017

Owner

I will close it. If something is not working consider reopen

Owner

tonytomov commented Dec 13, 2017

I will close it. If something is not working consider reopen

@tonytomov tonytomov closed this Dec 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment