Princeton-LSI-ResearchComputing / tracebase

Mouse Metabolite Tracing Data Repository for the Rabinowitz Lab
MIT License
4 stars 1 forks source link

Issue with empty space for table with a few rows using fixed table header #614

Open fkang-pu opened 1 year ago

fkang-pu commented 1 year ago

FEATURE REQUEST

Description

Using fixed table header with vertical scrolling is easier to view data in a table on a webpage, especially with many columns and/or rows. However, there are a few issues when using this feature with Bootstrap-table plugin. One issue is empty space for the table with vertical scrollbar when the table has fewer rows than pagination size.

The fixed table header feature requires the setting of table height in px.

Comment


DESIGN

Code Change Description

The new solution is based on the observation that table rendering goes through at least two steps by Bootstrap-table. Still need to dive deeper to understand the whole process.

Tests

hepcat72 commented 9 months ago

While working on other issues, I noted that the Infusates table pagination controls is off the bottom of the page unnecessarily (when you're scrolled to the top and there are only a handful of records). I think that's what this issue describes, but I'm not sure. In any case, I though I would post a screenshot that demonstrates the malformed page:

infusates_table_size_pushes_pagination_off_page
hepcat72 commented 9 months ago

Note how there is a vertical scrollbar that must be used to get to the pagination controls below the oversized table.