From 4daad6abed40625ccb45813a8a17047dd388535c Mon Sep 17 00:00:00 2001
From: jberg <jberg@users.noreply.github.com>
Date: Fri, 11 Oct 2019 16:15:53 -0700
Subject: [PATCH] Fix skin select box value (#940)

* fix skin select box value

* move skinUrl into local component state
---
 modern/src/App.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/modern/src/App.js b/modern/src/App.js
index b4209e3a..8a94e944 100644
--- a/modern/src/App.js
+++ b/modern/src/App.js
@@ -663,8 +663,10 @@ function Modern() {
   const dispatch = useDispatch();
   const store = useStore();
   const root = useSelector(Selectors.getMakiTree);
+  const [skinUrl, setSkinUrl] = React.useState(null);
   React.useEffect(() => {
     const defaultSkinUrl = getSkinUrlFromQueryParams() || skinUrls[0];
+    setSkinUrl(defaultSkinUrl);
     dispatch(Actions.gotSkinUrl(defaultSkinUrl, store));
   }, [store]);
   if (root == null) {
@@ -682,8 +684,10 @@ function Modern() {
       </DropTarget>
       <select
         style={{ position: "absolute", bottom: 0 }}
+        value={skinUrl}
         onChange={e => {
           const newSkinUrl = e.target.value;
+          setSkinUrl(newSkinUrl);
           // TODO: This should really go in a middleware somewhere.
           setSkinUrlToQueryParams(newSkinUrl);
           dispatch(Actions.gotSkinUrl(newSkinUrl, store));