The event listeners for "focus" and "click" in the Dashboard have not been properly removed

hkb1990 · December 30, 2025, 2:24am

packages/%40uppy/dashboard/src/Dashboard.tsx

23186da45


      
          
            this.uppy.off('plugin-added', this.#addSupportedPluginIfNoTarget)
            this.uppy.off('plugin-remove', this.removeTarget)
            this.uppy.off('file-added', this.hideAllPanels)
            this.uppy.off('dashboard:modal-closed', this.hideAllPanels)
            this.uppy.off('complete', this.handleComplete)
          
            this.uppy.off('files-added', this.#generateLargeThumbnailIfSingleFile)
            this.uppy.off('file-removed', this.#generateLargeThumbnailIfSingleFile)
          
            document.removeEventListener('focus', this.recordIfFocusedOnUppyRecently)
            document.removeEventListener('click', this.recordIfFocusedOnUppyRecently)
          
            if (this.opts.inline) {
              this.el!.removeEventListener('keydown', this.handleKeyDownInInline)
            }
          
            if (this.opts.autoOpen) {
              this.uppy.off('files-added', this.#openFileEditorWhenFilesAdded)
            }
          }

should be

document.removeEventListener(‘focus’, this.recordIfFocusedOnUppyRecently, true)
document.removeEventListener(‘click’, this.recordIfFocusedOnUppyRecently, true)

hkb1990 · December 31, 2025, 2:05am

PR created

Topic		Replies	Views
Is it possible to catch the trigger that opened the dashboard? Uppy	4	2161	September 17, 2017
Trigger file browser after reset Uppy	0	1229	March 8, 2021
Documentation update needed - uppy.requestCloseModal is not a function Uppy	0	595	January 8, 2023
Catch event when changing plugins Uppy	2	504	July 22, 2019
Disable browse file / drag and drop on uppy dashboard Uppy	2	256	April 28, 2025

The event listeners for "focus" and "click" in the Dashboard have not been properly removed

Related topics