CRDCDH-538 Fix Node Sorting & Partially-Visible Carousel Elements #288
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.
Overview
This is a minor change to the CRDCDH-538 implementation to fix an issue noticed in the prod builds.
Issue 1: Inconsistent Node Sorting
Without React StrictMode, the nodes in the primary chart would appear in the order that the API sent the nodes until you switch the view type.
Issue 2: Partially Visible Nodes
For small datasets (under 3 nodes), the carousel would partially overlap with the individual node charts and not provide arrows to make them visible. This is fixed now.
Cases:
Change Details (Specifics)
PaddingBox
element to the carousel childrenRelated Ticket(s)
CRDCDH-538