-
Notifications
You must be signed in to change notification settings - Fork 296
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 last column border is missing #54
Comments
This is related to sticky header scrollbar issues. The correct solution right now seems to be to make a change in carbon components styles to apply the border on |
We're definitely going to need the sticky headers even though probably will
have to do a custom implementation of it since we have very specific
requirements. I think the temp solution for now should be to add a
parameter that would turn off that additional column. I have a "no data"
card below my table and the width of the table and the card do not align:
![image](https://user-images.githubusercontent.com/1253469/45496651-f1aae200-b743-11e8-8773-619e7232a3a6.png)
Cheers,
Stan.
…On Wed, Sep 12, 2018 at 1:14 PM Zvonimir Fras ***@***.***> wrote:
This is related to sticky header scrollbar issues.
The correct solution right now seems to be to make a change in carbon
components styles to apply the border on thead instead of the last th.
Especially since sticky headers is a feature that we would, ideally,
support.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABMgXfAaGuxFbluocKRrA2__lju7mfS9ks5uaUD_gaJpZM4WjhHU>
.
|
@stanislavgeorgiev your image doesn't show for me here. |
Thanks @zvonimirfras, I reattached it to the post above. I attached it in my email reply but I guess github couldn't retrieve it from there. |
The other really annoying side effect from that hidden column is that my table ends up not centred in its box - left side is ok but there's more empty space on the right. |
Submitted a new issue with carbon-components: |
@zvonimirfras I'm closing the issue since |
On a second thought, we should adopt the dependency update first - it's currently using |
Bumped to latest in dev dependencies. 👍 |
This behavior can been seen in both my project and the library storybook:
It is caused by having an additional column automatically added by the table component itself:
The text was updated successfully, but these errors were encountered: