-
Notifications
You must be signed in to change notification settings - Fork 79
Closed
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.ArcGIS Maps SDK for JavaScriptIssues logged by ArcGIS SDK for JavaScript team members.Issues logged by ArcGIS SDK for JavaScript team members.bugBug reports for broken functionality. Issues should include a reproduction of the bug.Bug reports for broken functionality. Issues should include a reproduction of the bug.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 1Very small fix or change (potentially a single line), doesn't require updates to tests.Very small fix or change (potentially a single line), doesn't require updates to tests.impact - p1 - need for current milestoneUser set priority impact status of p1 - need for current milestoneUser set priority impact status of p1 - need for current milestonep - highIssue should be addressed in the current milestone, impacts component or core functionalityIssue should be addressed in the current milestone, impacts component or core functionality
Milestone
Description
Check existing issues
- I have checked for existing issues to avoid duplicates
Actual Behavior
Memory leak when adding/removing chips.
via @ycabon
Expected Behavior
Chips properly clean up after being removed from the DOM.
Reproduction Sample
https://codepen.io/jcfranco/pen/JjexPVj
Reproduction Steps
- open devtools memory tab
- click "add chip" then "remove chip" to warm up all the statics and what not
- take a memory snapshot
- click "add chip" then "remove chip"
- take another snapshot
- compare snapshot 2 with 1
Reproduction Version
1.4.3
Relevant Info
No response
Regression?
No response
Priority impact
p1 - need for current milestone
Impact
No response
Calcite package
- @esri/calcite-components
- @esri/calcite-components-react
Esri team
ArcGIS Maps SDK for JavaScript
Metadata
Metadata
Assignees
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.ArcGIS Maps SDK for JavaScriptIssues logged by ArcGIS SDK for JavaScript team members.Issues logged by ArcGIS SDK for JavaScript team members.bugBug reports for broken functionality. Issues should include a reproduction of the bug.Bug reports for broken functionality. Issues should include a reproduction of the bug.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 1Very small fix or change (potentially a single line), doesn't require updates to tests.Very small fix or change (potentially a single line), doesn't require updates to tests.impact - p1 - need for current milestoneUser set priority impact status of p1 - need for current milestoneUser set priority impact status of p1 - need for current milestonep - highIssue should be addressed in the current milestone, impacts component or core functionalityIssue should be addressed in the current milestone, impacts component or core functionality