filamentgroup / tablesaw

A group of plugins for responsive tables.
MIT License
5.48k stars 434 forks source link

When both pagination arrows are disabled, add an additional class so we can hide the arrows #198

Closed toddparker closed 8 years ago

toddparker commented 8 years ago

On wider screens, we may not need pagination arrows for the columns because the table fits. Right now, the plugin will correctly disable both arrows but this can be confusing to a user. It would be preferable to hide these until at least one of them is needed and this can be done by adding a class to the parent of the pagination buttons such as no-pagination-needed that would let us add a rule to hide or style them in a different way in this condition.

zachleat commented 8 years ago

Fixed with https://github.com/filamentgroup/tablesaw/commit/0dc86538911e5bdfc814e7b465f8dcd86843db46