From 06eda75be4e578fd020361a29bc1a6700d3ae1de Mon Sep 17 00:00:00 2001 From: Artur Paikin Date: Wed, 19 Sep 2018 19:10:36 -0400 Subject: [PATCH] call `hideAllPanels` on `file-added` I believe this was removed because we were doing `addFile` right away when it was selected in `Providers`, which is not the case now, so safe to close Dashboard panels when a file is added. --- packages/@uppy/dashboard/src/index.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/@uppy/dashboard/src/index.js b/packages/@uppy/dashboard/src/index.js index 375ef011d..a8f337908 100644 --- a/packages/@uppy/dashboard/src/index.js +++ b/packages/@uppy/dashboard/src/index.js @@ -456,7 +456,10 @@ module.exports = class Dashboard extends Plugin { this.ro.observe(this.el.querySelector('.uppy-Dashboard-inner')) this.uppy.on('plugin-remove', this.removeTarget) - this.uppy.on('file-added', (ev) => this.toggleAddFilesPanel(false)) + this.uppy.on('file-added', (ev) => { + this.toggleAddFilesPanel(false) + this.hideAllPanels() + }) } removeEvents () { @@ -658,7 +661,9 @@ module.exports = class Dashboard extends Plugin { const plugins = this.opts.plugins || [] plugins.forEach((pluginID) => { const plugin = this.uppy.getPlugin(pluginID) - if (plugin) plugin.mount(this, plugin) + if (plugin) { + plugin.mount(this, plugin) + } }) if (!this.opts.disableStatusBar) {