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

floatThead-container width when scrollContainer overflow is hidden #128

Merged
merged 1 commit into from
Oct 15, 2014

Conversation

XAOPT
Copy link
Contributor

@XAOPT XAOPT commented Oct 14, 2014

I have two tables with synchronous scroll. One of them has hidden overflow-y.
Try to scroll right one (table2) vertically, and you'll see bug in table1's header.

http://jsfiddle.net/8cxe4qxc/
Screenshot: http://take.ms/Z5SBq

@mkoryak
Copy link
Owner

mkoryak commented Oct 14, 2014

What browser / os is this on?

@XAOPT
Copy link
Contributor Author

XAOPT commented Oct 15, 2014

Win7 Professional.
Chrome 37.0.2062.124 and 38.0.2125.104
Firefox 32.0.3
IE 11.0.9600.17239
Chrome 40.0.2189.0 canary

@mkoryak
Copy link
Owner

mkoryak commented Oct 15, 2014

ah, thats probably why I cant reproduce on my mac ;)

Ill check it out on a vm

mkoryak added a commit that referenced this pull request Oct 15, 2014
floatThead-container width when scrollContainer overflow is hidden
@mkoryak mkoryak merged commit e0bb475 into mkoryak:master Oct 15, 2014
mkoryak added a commit that referenced this pull request Oct 15, 2014
mkoryak added a commit that referenced this pull request Oct 15, 2014
@mkoryak
Copy link
Owner

mkoryak commented Oct 15, 2014

its on master, not going to cut a new version just yet, unless you really really need it with sugar on top

@mkoryak
Copy link
Owner

mkoryak commented Oct 15, 2014

mkoryak added a commit that referenced this pull request Oct 21, 2014
… happens when you init plugin on a hidden table and later show it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants