precision-sustainable-ag / dst-selector

https://covercrop-selector.org
MIT License
3 stars 1 forks source link

Fix - sticky table headers #657

Closed jinamshah closed 2 months ago

jinamshah commented 2 months ago

What done looks like

Where the changes need to be made

jinamshah commented 2 months ago

Sticky headers are not supported for elements without fixed heights, and since we're using grids, we couldn't have provided a fixed height without having two scroll bars - one for page and one for the table. This issue would create problems in all tables. Hence, the solution is as follows: Store the widths of the table and sidebar on a page and compare it with window size, if greater, then change the width of the header and footer to accommodate this change in window size. This + overflowX: 'initial' removes the table scrolls (both horizontal and vertical).

Also, crop card has been removed because it was taking too much room and instead crop's scientific name is added to the table and modal open/close is added to the crop label header

mikahpinegar commented 2 months ago

Your review app has been deployed and is available at 'http://135.237.20.24:80'.

Learn More about how to test and provide feedback for the app.

mikahpinegar commented 2 months ago

Your review app has been deployed and is available at 'http://135.237.20.24:80'.

Learn More about how to test and provide feedback for the app.

mikahpinegar commented 2 months ago

Your review app has been deployed and is available at 'http://135.237.20.24:80'.

Learn More about how to test and provide feedback for the app.

mikahpinegar commented 2 months ago

Your review app has been deployed and is available at 'http://135.237.20.24:80'.

Learn More about how to test and provide feedback for the app.