Move removeListener to unregister

So that it’s properly called when the DOMNode is removed from the DOM
missive · Jan 6, 2024 · 96eb041 · 96eb041
1 parent cda05ca
commit 96eb041
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/packages/emoji-mart/src/components/Picker/Picker.tsx b/packages/emoji-mart/src/components/Picker/Picker.tsx
@@ -106,7 +106,6 @@ export default class Picker extends Component {
 
   componentWillUnmount() {
     this.unregister()
-    this.darkMedia?.removeListener(this.darkMediaCallback)
   }
 
   async reset(nextState = {}) {
@@ -128,6 +127,7 @@ export default class Picker extends Component {
 
   unregister() {
     document.removeEventListener('click', this.handleClickOutside)
+    this.darkMedia?.removeListener(this.darkMediaCallback)
     this.unobserve()
   }