[GitHub] Georift commented on issue #1146: Add grouping to databases with IDs containing '/'

[GitBox <gi...@apache.org>]

Georift commented on issue #1146: Add grouping to databases with IDs containing '/'
URL: https://github.com/apache/couchdb-fauxton/pull/1146#issuecomment-432147837
 
 
   Ok I'll look into getting it converted to lodash and add multiple nesting when I get a chance.
   
   I tested this change with pagination, it doesn't change how the pages are generated and only groups those databases that are visible on the current page. I guess this could be a little confusing? As you can have a grouping that spans several pages and if it appears earlier in the list may push single database entries 10's of pages away. The intermediary pages look like this:
   ![entirely grouped page](https://user-images.githubusercontent.com/859430/47346546-ffa73780-d6df-11e8-8786-1f2cd7379fad.png)
   
   The easiest solution in my mind would be to push any groupings to the end of the sort so it never obscures single databases. The other option could be to do pagination on the groups themselves and only display a single entry on any of the top level pages. But that adds a ton more work I'd imagine.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services