TODO: GridField Component Improvements

[sabina-talipova]

Description

See:

silverstripe-admin/client/src/components/GridField/GridField.js

Lines 23 to 25 in 0f18b3d

	* @todo Convert to higher order component which hooks up form
	* schema data to an API backend as a grid data source
	* @todo Replace "dumb" inner components with third party library (e.g. https://griddlegriddle.github.io)

• Convert to higher order component which hooks up form schema data to an API backend as a grid data source
• Replace "dumb" inner components with third party library (e.g. https://griddlegriddle.github.io)
• Replace <div>{ i18n._t('CampaignAdmin.LOADING', 'Loading...') }</div> with better loading indicator. See

[GuySartorelli]

@sabina-talipova it's not clear what is meant by "dumb" inner components, and that griddle link is a 404 - can you please either clarify what the intention was or remove that point?

If that point ends up being removed then I think we can close this issue, as the other points aren't worth doing on their own IMO.

[sabina-talipova]

I assume that the idea was to replace createRow and createCells and so on, with this JS module - Griddle

[GuySartorelli]

I'm going to close this for now - we're not likely to spend time working on small parts of the js of gridfield.
I suspect next time we look at enhancing or improving the gridfield javascript, it will be more akin to a complete overhaul or replacement.