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

Empty nimble-table is missing horizontal scrollbar #2537

Open
mollykreis opened this issue Feb 10, 2025 · 0 comments
Open

Empty nimble-table is missing horizontal scrollbar #2537

mollykreis opened this issue Feb 10, 2025 · 0 comments
Labels
bug Something isn't working triage New issue that needs to be reviewed

Comments

@mollykreis
Copy link
Contributor

mollykreis commented Feb 10, 2025

🐛 Bug Report

The nimble table with no data can end up without a horizontal scrollbar even when all the columns do not fit within the width of the table.

For example, in the screenshot below the columns are clipped but no scrollbar is present:

Image

💻 Repro or Code Sample

🤔 Expected Behavior

When the columns don't all fit in the width of the nimble-table, a horizontal scrollbar should be present, regardless of whether or not there are any rows of data in the table.

😯 Current Behavior

The table can get into a state where a horizontal scrollbar isn't present when there is no data in the table.

To reproduce in nimble's storybook:

  1. Open a table story (not the docs page), such as https://nimble.ni.dev/storybook/?path=/story/components-table--table
  2. Reduce the width of your browser window to make the table have a horizontal scrollbar
  3. In dev tools, clear the data in the table
  4. Notice that the horizontal scrollbar disappears

💁 Possible Solution

🔦 Context

🌍 Your Environment

  • OS & Device: [e.g. MacOS, iOS, Windows, Linux] on [iPhone 7, PC]
  • Browser [e.g. Microsoft Edge, Google Chrome, Apple Safari, Mozilla FireFox]
  • Version [e.g. 1.8.0]
@mollykreis mollykreis added bug Something isn't working triage New issue that needs to be reviewed labels Feb 10, 2025
@m-akinc m-akinc added triage New issue that needs to be reviewed and removed triage New issue that needs to be reviewed labels Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage New issue that needs to be reviewed
Projects
Status: No status
Development

No branches or pull requests

2 participants