Skip to content
This repository has been archived by the owner on Dec 12, 2018. It is now read-only.

Placeholder Rows #9

Open
pottedmeat opened this issue Feb 27, 2017 · 1 comment
Open

Placeholder Rows #9

pottedmeat opened this issue Feb 27, 2017 · 1 comment

Comments

@pottedmeat
Copy link
Contributor

Enhancement

When interacting with the grid, we will often see a row that spans all columns that contains a simple message about the behavior taking place. In dgrid 1, the following overrides appear (as strings):

  • loadingMessage
  • noDataMessage
  • errorMessage

As an attempt to cover all situations placeholder rows may appear, consider the following:

  • Loading message: when a row has been scrolled to but the data for that row has not yet been loaded, in the case of very large data sets
  • Loading message: when expanding a tree, a single row may be added showing data is being loaded with information about what is being loaded or multiple loading rows showing the data for that row has not yet been loaded
  • Error: when an asynchronous operation fails, it could add an interactive row to trigger a retry, or outline a catastrophic error in the case of being offline
  • No data: when filtering/searching results in no data being found

The discussion we should have is whether we can outline every situation we expect to encounter during grid interaction and provide widgets for these, exposed through the grid's registry or introduce a completely dynamic approach where we ask a callback for the widget to use based on an item the data provider placed in the data set.

@dylans
Copy link
Member

dylans commented Feb 28, 2017

Presumably the placeholder rows will just be widgets, and they can existing as something that covers an entire dgrid, the entire body of the grid (excluding headers/footers), a row, or a cell (or any other widget level in a grid). Beyond that they contain a series of messages, some standard, and perhaps some that are custom). I can conceive a scenario where dismissing them could trigger a change event. For example, suppose the data was empty because of a network timeout or the data becomes stale (the latter of which is perhaps another message to have for a placeholder?), and the user wants to refresh/try again?

cc @matt-gadd and @agubler for their thoughts.

@eheasley eheasley added the beta2 label Mar 15, 2017
@dylans dylans modified the milestones: 2017.03, 2017.04 Apr 2, 2017
@eheasley eheasley added beta3 and removed beta2 labels Apr 6, 2017
@dylans dylans modified the milestones: 2017.05, 2017.04 Apr 8, 2017
@eheasley eheasley modified the milestones: 2017.05, 2017.06 Jun 6, 2017
@dylans dylans modified the milestones: 2017.06, 2017.07 Jul 4, 2017
@dylans dylans modified the milestones: 2017.07, 2017.08 Jul 29, 2017
@kitsonk kitsonk modified the milestones: 2017.08, 2017.09 Sep 4, 2017
@kitsonk kitsonk removed the beta3 label Oct 9, 2017
@kitsonk kitsonk modified the milestones: 2017.09, 2017.11 Oct 9, 2017
@kitsonk kitsonk removed this from the 2017.11 milestone Dec 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants