search

casbin / casbin-hub

The hub for Casbin
https://dashboard.casbin.com
Apache License 2.0
87 stars 46 forks source link

Nested scrollbars in the app #24

Closed nik72619c closed 4 years ago

nik72619c commented 4 years ago

Current behaviour

When a list of items become large, a scrollbar appears for that list item . This creates a somewhat bad UX according to me.

Also, since pagination is applied here I feel that the scrollbar for should not be there at all

@hsluoyz Please let me know if I shall take this forward :) Working on this one

hsluoyz commented 4 years ago

Show pictures.

techguybiswa commented 4 years ago

Hi @hsluoyz,

I am attaching a picture of what I observed.

So as you can see when I add a large number of rows the table becomes scrollable. Also, we have an. already existing pagination. And the pagination does not work.

So we can do one of these:

  1. Remove pagination and make it scrollable (not so great UX)
  2. Or remove scroll and make it paginated after with like 10 rows per page.
  3. Or Implement pagination along with scroll.

A screen recording : https://www.loom.com/share/a05e5202d0cc4830bf0d5b37a55e6ae3 If nothing is done then the UI/UX, in general, seems confusing.

If this is a valid issue then I can work on it.

hsluoyz commented 4 years ago

Or remove scroll and make it paginated after with like 10 rows per page.

This would be good.