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

[Segment/Menu/Table] - Fix Doubled Borders in Some Attached Cases #2599

Closed
celsworth opened this issue Jul 12, 2015 · 1 comment
Closed

[Segment/Menu/Table] - Fix Doubled Borders in Some Attached Cases #2599

celsworth opened this issue Jul 12, 2015 · 1 comment
Milestone

Comments

@celsworth
Copy link

I noticed that my tables had a slightly different width from their attached menus while in a grid, see http://jsfiddle.net/zsu3k0nt/1/

It only seems to happen in a grid, and might be because the table width calculation is "less accurate" than the others? For example, looking at it now and I have a width of 260.750px on the segment and menu, but 260.000px on the table, which seems to end up getting rounded differently as follows:

You might have to change the width of the preview panel to see it in action.

@jlukic
Copy link
Member

jlukic commented Jul 12, 2015

The borders should not be doubled between segment->menu and menu->table when attached. I need to do more tests with mixed component attachment.

With regard to width being off by a partial amount on tables. That could be related to rounding with table, but I can't verify. Maybe table-layout: fixed might behave differently.

@jlukic jlukic added this to the 2.0.4 milestone Jul 12, 2015
@jlukic jlukic changed the title Width calculation for tables differs from other elements? [Segment/Menu/Table] - Fix Doubled Borders in Some Attached Cases Jul 12, 2015
jlukic added a commit that referenced this issue Jul 17, 2015
@jlukic jlukic closed this as completed Jul 17, 2015
jlukic added a commit that referenced this issue Jul 17, 2015
jlukic added a commit that referenced this issue Jul 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants