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

Clone header overlaps handsontable cells #4787

Closed
jarben opened this issue Feb 8, 2018 · 5 comments
Closed

Clone header overlaps handsontable cells #4787

jarben opened this issue Feb 8, 2018 · 5 comments

Comments

@jarben
Copy link

@jarben jarben commented Feb 8, 2018

When a cell is rendered as handsontable or autocomplete, a second HOT instance overlaps popup menu:

image

Demo: http://jsfiddle.net/jarben/c2szqw0h/
Version: 0.35.1
OS: win

@AMBudnik

This comment has been minimized.

Copy link
Contributor

@AMBudnik AMBudnik commented Feb 9, 2018

Thank you for sharing. I have replicated the issue.
As I workaround I recommend to bump up the z-index of the handsontableInputHolder class
Updated demo http://jsfiddle.net/gspxpq1s/

@jarben

This comment has been minimized.

Copy link
Author

@jarben jarben commented Feb 9, 2018

Cool, thanks for a workaround @AMBudnik !

@AMBudnik

This comment has been minimized.

Copy link
Contributor

@AMBudnik AMBudnik commented Feb 9, 2018

You're welcome @jarben
Now, that we know how the workaround it would be easier to fix it in the official version.

@pnowak pnowak self-assigned this Mar 2, 2018
pnowak added a commit that referenced this issue Mar 2, 2018
… z-index handsontableInputHolder #4787
@pnowak pnowak mentioned this issue Mar 2, 2018
2 of 5 tasks complete
pnowak added a commit that referenced this issue Mar 8, 2018
@pnowak pnowak added this to the April 2018 milestone Mar 13, 2018
@pnowak

This comment has been minimized.

Copy link
Member

@pnowak pnowak commented Mar 15, 2018

The old demos won't work on newer version of Handsontable (a consequence of selection API change). Updated demo - http://jsfiddle.net/safva57x/

@AMBudnik

This comment has been minimized.

Copy link
Contributor

@AMBudnik AMBudnik commented Apr 11, 2018

Hi @jarben

please update to v 2.0.0. (CE and PRO) and let me know if it's fixed for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.