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

Table getting stuck at width when resizing horizontally. #70

Closed
Kilowhisky opened this issue Mar 26, 2014 · 9 comments
Closed

Table getting stuck at width when resizing horizontally. #70

Kilowhisky opened this issue Mar 26, 2014 · 9 comments
Labels

Comments

@Kilowhisky
Copy link

When working with responsive layout in bootstrap 3 the table gets stuck at a small width and will not resize with 'reflow' or 'destroy'.

I noticed that the problem does not occur if i remove the attribute 'width:auto' that is attached to the table style.

Browser: Chrome (newest)
Example: http://mkoryak.github.io/floatThead/examples/bootstrap3/
Replicate:
1: Scroll vertically to a point where a table header is floating above a row
2: Resize the window horizontally to a small size.
3: Resize the window back to a large width.

@mkoryak
Copy link
Owner

mkoryak commented Mar 27, 2014

Thanks, looks like all my recent meddling with how table sizing works had this unfortunate side-effect. Ill take a look tomorrow.

@mkoryak mkoryak added the bug label Mar 27, 2014
mkoryak added a commit that referenced this issue Mar 31, 2014
@mkoryak
Copy link
Owner

mkoryak commented Mar 31, 2014

Can you check your code against the v1.2.6 branch and see if this is resolved there?

@Kilowhisky
Copy link
Author

It appears to be fixed, thanks! Although i will say it is quite slow resizing on chrome and IE 11, Firefox is fast though.

EDIT: found one issue which is not really that bad but can cause horizontal scroll bars even when they are not needed. It looks like the colgroup style width doesn't account for the like 1-3 pixels or table border. I'm not so sure but if i remove the colgroup element the scroll bars are gone.

@mkoryak
Copy link
Owner

mkoryak commented Apr 1, 2014

can you create a fiddle with the border issue so i can see it happening?
colgroup element is required for the plugin to work, so removing it might be bad ;)

@Kilowhisky
Copy link
Author

Is there a cdn for the 1.2.6 branch?

@mkoryak
Copy link
Owner

mkoryak commented Apr 1, 2014

@Kilowhisky
Copy link
Author

http://jsfiddle.net/w9UNJ/

perform the same test as described in the first post.

mkoryak added a commit that referenced this issue Apr 2, 2014
made the issue html nicer
@mkoryak
Copy link
Owner

mkoryak commented Apr 2, 2014

the problem seems to be unrelated to the plugin, see:
http://jsfiddle.net/w9UNJ/2/

I removed all javascript code and the scrollbars still appear.

@mkoryak mkoryak added question and removed bug labels Apr 3, 2014
@mkoryak mkoryak closed this as completed Apr 8, 2014
mkoryak added a commit that referenced this issue Apr 8, 2014
* origin/v1.2.6:
  ooops, this is the issue this fixes #70
  fix #71 - setting table to width auto is not actually as good as letting the width be whatever it should be via css (that we dont know about)
@lock
Copy link

lock bot commented Dec 11, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants