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] fixed with width: 100% is ignoring scrollbar width #2785

Closed
mvorisek opened this issue May 12, 2023 · 2 comments
Closed

[table] fixed with width: 100% is ignoring scrollbar width #2785

mvorisek opened this issue May 12, 2023 · 2 comments
Labels
type/usage Any support issues asking for help

Comments

@mvorisek
Copy link
Contributor

Bug Report

See the repro, with fixed the table data are set to 100% width, but the data on the right side are covered by a scrollbar.

Steps to reproduce

  1. open https://dev.atk4.org/demos/interactive/scroll-grid-container.php

Expected result

no data covered/hidden by anything

Actual result

image

Version

latest + Firefox

@mvorisek mvorisek added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels May 12, 2023
@lubber-de
Copy link
Member

In your use case the column just isnt wide enough. As the docs say fixed uses table-layout which does not adjust the columns by its content. Your example would not even display the other missing button inside the last column.

To solve this add a propriate wide class to the last column, so it gets a fixed width.
Sidenote: Adding scrolling would also remove the need to have the fake extra header

See https://jsfiddle.net/lubber/807q5h9p/13/

Of course if the the whole table gets too small but also has too many columns at the same time (and also should not stack) the whole table layout inside the related app should be thought about again.

@lubber-de lubber-de added type/usage Any support issues asking for help and removed type/bug Any issue which is a bug or PR which fixes a bug state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged labels May 12, 2023
@mvorisek
Copy link
Contributor Author

Sidenote: Adding scrolling would also remove the need to have the fake extra header

separated header is never good, thank you for the native scrolling hint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/usage Any support issues asking for help
Projects
None yet
Development

No branches or pull requests

2 participants