Make long lists of demographic unknowns collapsible #448
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the change
Particularly with time series data, our treatment of people with unknown demographic properties can get pretty unwieldy if there are lots of them. We never had this problem in ND and so we never solved for it, but it does appear in the PA parole data. This just clamps the list at 3 entries with a button to expand or collapse it in place as needed. (With 3 or fewer entries the button will not be shown.)
The height change is animated with a technique I've used in several other places. Rather than duplicate it yet another time I have refactored it into a wrapper component. I haven't seen any regression in those sections. (This does make the PR look bigger than it really is due to indentation changes; you might want to hide them while reviewing.)
Type of change
Related issues
Checklists
Development
These boxes should be checked by the submitter prior to merging:
Code review
These boxes should be checked by reviewers prior to merging: