Merge branch 'issue-621-v' into develop

Author: callmekatootie

client/src/components/AddToGroupModal/index.jsx

@@ -1,5 +1,5 @@
 import React, { useEffect } from "react";
-import { List } from "react-virtualized";
+import { List, AutoSizer } from "react-virtualized";
 import PT from "prop-types";
 
 import Button from "../Button";
@@ -237,24 +237,27 @@ export default function AddToGroupModal({ onCancel, updateUser, user }) {
         <h3 className={style.subTitle}>
           My groups{loadingGroups && " (Loading...)"}
         </h3>
-        <div>
-          {!loadingGroups && (
-            <List
-              className={style.groupsList}
-              width={listWidth}
-              height={
-                filteredMyGroups.length > 10
-                  ? 450
-                  : filteredMyGroups.length * 45
-              }
-              rowCount={filteredMyGroups.length}
-              rowHeight={45}
-              rowRenderer={(params) =>
-                rowRenderer({ ...params, items: filteredMyGroups })
-              }
-            />
-          )}
-        </div>
+
+        {!loadingGroups && (
+          <AutoSizer disableHeight>
+            {() => (
+              <List
+                className={style.groupsList}
+                width={listWidth + 20}
+                height={
+                  filteredMyGroups.length > 10
+                    ? 250
+                    : filteredMyGroups.length * 45
+                }
+                rowCount={filteredMyGroups.length}
+                rowHeight={45}
+                rowRenderer={(params) =>
+                  rowRenderer({ ...params, items: filteredMyGroups })
+                }
+              />
+            )}
+          </AutoSizer>
+        )}
         {myGroups.filter((g) =>
           g.name.toLowerCase().includes(filter.toLowerCase())
         ).length === 0 &&
@@ -264,22 +267,26 @@ export default function AddToGroupModal({ onCancel, updateUser, user }) {
         <h3 className={style.subTitle}>
           Other Groups{loadingGroups && " (Loading...)"}
         </h3>
-        <div>
+        <div className={style.clear}>
           {!loadingGroups && (
-            <List
-              className={style.groupsList}
-              width={listWidth}
-              height={
-                filteredOtherGroups.length > 10
-                  ? 450
-                  : filteredOtherGroups.length * 45
-              }
-              rowCount={filteredOtherGroups.length}
-              rowHeight={45}
-              rowRenderer={(params) =>
-                rowRenderer({ ...params, items: filteredOtherGroups })
-              }
-            />
+            <AutoSizer disableHeight>
+              {() => (
+                <List
+                  className={style.groupsList}
+                  width={listWidth + 20}
+                  height={
+                    filteredOtherGroups.length > 10
+                      ? 250
+                      : filteredOtherGroups.length * 45
+                  }
+                  rowCount={filteredOtherGroups.length}
+                  rowHeight={45}
+                  rowRenderer={(params) =>
+                    rowRenderer({ ...params, items: filteredOtherGroups })
+                  }
+                />
+              )}
+            </AutoSizer>
           )}
         </div>
         {otherGroups.filter((g) =>

client/src/components/AddToGroupModal/style.module.scss

@@ -78,7 +78,7 @@
 .searchRow {
   display: flex;
   position: relative;
-  margin: 0 30px;
+  margin: 0 10px;
 }
 
 .subTitle {
@@ -123,3 +123,8 @@
     visibility: hidden;
   }
 }
+
+.clear {
+  padding-right: 15px;
+  overflow: auto;
+}

client/src/components/Modal/style.module.scss

@@ -27,7 +27,7 @@
   max-width: $screen-md;
   overflow: hidden;
   padding: 30px;
-  min-width: 517px;
+  min-width: 520px;
   position: fixed;
   top: 50%;
   left: 50%;