From 1f329174ce140e323eafd9dc8c2b6e573e02aa7f Mon Sep 17 00:00:00 2001 From: Ricky Miller Date: Mon, 12 Feb 2018 09:06:48 -0600 Subject: [PATCH] Add optional accept argument --- js/actionCreators.js | 4 ++-- js/components/MainWindow/MainContextMenu.js | 2 +- js/fileUtils.js | 5 ++++- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/js/actionCreators.js b/js/actionCreators.js index 759468e9..7c5efc4c 100644 --- a/js/actionCreators.js +++ b/js/actionCreators.js @@ -339,9 +339,9 @@ export function setSkinFromUrl(url) { }; } -export function openFileDialog() { +export function openFileDialog(accept) { return async dispatch => { - const fileReferences = await promptForFileReferences(); + const fileReferences = await promptForFileReferences(accept); dispatch(loadFilesFromReferences(fileReferences)); }; } diff --git a/js/components/MainWindow/MainContextMenu.js b/js/components/MainWindow/MainContextMenu.js index 8d65eab2..cd375089 100644 --- a/js/components/MainWindow/MainContextMenu.js +++ b/js/components/MainWindow/MainContextMenu.js @@ -18,7 +18,7 @@ const MainContextMenu = props => (
- + {!!props.avaliableSkins.length &&
} {props.avaliableSkins.map(skin => ( { // Does this represent a memory leak somehow? // Can this fail? Do we ever reject? const fileInput = document.createElement("input"); + if (accept) { + fileInput.setAttribute('accept', accept) + } fileInput.type = "file"; fileInput.multiple = true; // Not entirely sure why this is needed, since the input