Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Data parameter for grids #112

Open
gskinner opened this issue May 6, 2021 · 1 comment
Open

Data parameter for grids #112

gskinner opened this issue May 6, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@gskinner
Copy link
Collaborator

gskinner commented May 6, 2021

It would be nice to add a data parameter for grids, so that those values can be set externally. One consideration is what to do with the default value – including the full list of items seems excessive, but it would be valuable to include at least one item to make it easy to reference the fields. This is complicated by the current requirement to define parameters in the constructor instead of during serialization.
@gskinner gskinner added the enhancement New feature or request label May 6, 2021
gskinner added a commit that referenced this issue May 6, 2021
@gskinner
Initial implementation of grid data params
b212af7 
per #112
@gskinner
Copy link
Collaborator Author

gskinner commented May 6, 2021

This is partially complete, but currently includes an empty default. Still needs updating to include the first item as the default. Requires testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gskinner