fix: "update all mods" button and pages width

blarfoon · blarfoon · commit 1d17f1287a5b · 2022-07-05T12:50:20.000+02:00
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -156,7 +156,7 @@
     "@sentry/tracing": "^6.16.1",
     "@sentry/utils": "^6.16.1",
     "7zip-bin": "^5.1.1",
-    "antd": "^4.21.2",
+    "antd": "^4.21.5",
     "axios": "^0.27.2",
     "base64-stream": "^1.0.0",
     "base64url": "^3.0.1",
diff --git a/src/app/desktop/components/Instances/index.js b/src/app/desktop/components/Instances/index.js
@@ -8,7 +8,6 @@ const Container = styled.div`
   display: flex;
   flex-wrap: wrap;
   width: 100%;
-  justify-content: center;
   margin-bottom: 2rem;
 `;
 
diff --git a/src/app/desktop/utils/withScroll.js b/src/app/desktop/utils/withScroll.js
@@ -17,7 +17,11 @@ const withScroll = Component => {
             justify-content: center;
           `}
         >
-          <div>
+          <div
+            css={`
+              width: 1000px;
+            `}
+          >
             {/* eslint-disable-next-line react/jsx-props-no-spreading */}
             <Component {...props} />
           </div>
diff --git a/src/common/modals/InstanceManager/Mods.js b/src/common/modals/InstanceManager/Mods.js
@@ -690,9 +690,9 @@ const Mods = ({ instanceName }) => {
       items={[
         {
           key: '0',
+          disabled: !hasModUpdates,
           label: (
             <div
-              disabled={!hasModUpdates}
               onClick={() => {
                 dispatch(openModal('ModsUpdater', { instanceName }));
                 setIsMenuOpen(false);
@@ -764,6 +764,8 @@ const Mods = ({ instanceName }) => {
                 try {
                   setLoadingModUpdates(true);
                   await dispatch(initLatestMods(instance.name));
+                } catch (e) {
+                  console.warn(e);
                 } finally {
                   setLoadingModUpdates(false);
                 }
@@ -776,27 +778,14 @@ const Mods = ({ instanceName }) => {
           <span
             onClick={e => {
               e.stopPropagation();
-              setIsMenuOpen(!isMenuOpen);
+              setIsMenuOpen(prev => !prev);
             }}
           >
-            <StyledDropdown
-              onClick={e => {
-                if (!isMenuOpen) {
-                  e.stopPropagation();
-                  setIsMenuOpen(!isMenuOpen);
-                }
-              }}
-            >
-              <span
-                onClick={e => {
-                  e.stopPropagation();
-                  setIsMenuOpen(!isMenuOpen);
-                }}
-              >
+            <StyledDropdown>
+              <span>
                 <Dropdown
                   overlay={menu}
                   visible={isMenuOpen}
-                  onVisibleChange={setIsMenuOpen}
                   trigger={['click']}
                 >
                   <span
