[Resolve Errors] [Review Warnings] Tables - selectively enable borders based on if the table is overflowing

[shindigira]

• useIsOverflowing hook already added to the app
• Requires ref forwarding on the Table container

[natalia-fitzgerald]

@shindigira @meissadia
What if instead of doing this we removed the top, left, and right borders from the scrollable tables? I would want to only include the full border on the last page of paginated tables.

[shindigira]

@shindigira @meissadia What if instead of doing this we removed the top, left, and right borders from the scrollable tables? I would want to only include the full border on the last page of paginated tables.

@natalia-fitzgerald What is the main UX aspect we are trying to convey?

Having a certain borders styling which will emphasize that the table is overflowing and thus horizontal scrolling is enabled?

[natalia-fitzgerald]

Yes, that is the aspect that any visual styling difference would be meant to convey. This is why it isn't ideal when tables that don't require horizontal scrolling receive this bordered styling. Generally I prefer the open styling of no borders (standard table styling). Since our app has the need for some horizontal scrolling tables we end up with visual inconsistency on the page. I'd like to have the horizontal scroll styling (borders) affect only the tables that require it.