feat: improved position of modal (#19)

* feat: improved position of modal - Overrode modal CSS to be towards top of page - Changed page limit to 10 so you don't have to scroll after the modal * chore: nit
edx · Dec 5, 2023 · 9500a81 · 9500a81
1 parent 466c54a
commit 9500a81
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/src/pages/ExamsPage/components/AttemptList.jsx b/src/pages/ExamsPage/components/AttemptList.jsx
@@ -110,7 +110,7 @@ const AttemptList = ({ attempts }) => {
         isLoading={attempts == null}
         isPaginated
         initialState={{
-          pageSize: 20,
+          pageSize: 10,
         }}
         isFilterable
         defaultColumnValues={{ Filter: TextFilter }}

diff --git a/src/pages/ExamsPage/components/ResetExamAttemptButton.jsx b/src/pages/ExamsPage/components/ResetExamAttemptButton.jsx
@@ -23,7 +23,7 @@ const ResetExamAttemptButton = ({ username, examName, attemptId }) => {
         title="my dialog"
         isOpen={isOpen}
         onClose={close}
-        size="md"
+        size="lg"
         variant="default"
         hasCloseButton
         isFullscreenOnMobile

diff --git a/src/pages/ExamsPage/components/ReviewExamAttemptButton.jsx b/src/pages/ExamsPage/components/ReviewExamAttemptButton.jsx
@@ -76,7 +76,7 @@ const ReviewExamAttemptButton = ({
         title="my dialog"
         isOpen={isOpen}
         onClose={close}
-        size="md"
+        size="lg"
         variant="default"
         hasCloseButton
         isFullscreenOnMobile

diff --git a/src/pages/ExamsPage/index.scss b/src/pages/ExamsPage/index.scss
@@ -12,3 +12,9 @@
   border: none;
   width: 0px;
 }
+
+// Align modals towards the top of the iframe, as it aligns center (which is awkward to have to scroll to) by default
+div.pgn__modal-content-container {
+  align-items: start;
+  padding-top: 6.5%;
+}