kos / rtable Goto Github PK

Currently there can reasonably be only one RTable on a page, and it completely dominates the URL's query string.

In order to have several of these on one page, one of the following needs to happen:

a. they no longer read from or update window.location. The table state might be retained somewhere else in this case (like the session storage).
b. they still work with window.location, but each has a separate prefix. The page URL contains parameters for each table on the page.

Both approaches could be implemented as opt-in.

Currently the ordering works on only one column at the time. Why?

Clicking on several headers in sequence should remember the state of previous headers clicked, giving the functional equivalent of stable sorting.

I want to be able to define:

• how does each row look like?
• how the pagination and filter elements look like?
• how is the whole RTable pieced together?

The idea I have is to break down RTable.render() so that it doesn't produce any HTML on its own, but rather delegates pieces of its own state to sub-components that can be replaced individually.

The default sub-components should also offer some more flexibility, esp. the class names used, so that it's easy to make it "look like bootstrap" or "look like semantic ui" without actually redefining the react components.

The code assumes a lot about page numbers - that they are integers and the initial page is 1.

When possible, the page number should be treated as an opaque identifier, so that you can paginate using cursors as well as page numbers. Page numbers shouldn't be generally produced by RTable, only taken from incoming DataResponses.

However, we'd still want to allow extra features that base on numerical pages, like "showing page 5 of 123" or "go to page 5". This might mean that RTable would have a separate mode for numerical and opaque page identifiers. Or we could just look at DataResponse and decide what to show (as in, if we have count and the page identifier is numeric, then assume sequential pages)