Skip to content

Commit

Permalink
Remove btm spacer, fix scroll bars in modals (#1053)
Browse files Browse the repository at this point in the history
Since we decided that our modals should be
scrollable, some of them always have a
scrollbar showing, due to an absolutely
positioned white box at the very bottom of
the modal. I am not sure what the original
purpose of this "btm-spacer" component
was supposed to be, but it seems superfluous
to me and so I am removing it. This should stop
the scrollbars from appearing unnecessarily.
  • Loading branch information
Arnei authored Feb 11, 2025
1 parent edbcbec commit 75ddd63
Show file tree
Hide file tree
Showing 17 changed files with 1 addition and 68 deletions.
2 changes: 0 additions & 2 deletions src/components/configuration/partials/wizard/ThemeDetails.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,6 @@ const ThemeDetails = ({
createTranslationString={"SUBMIT"}
cancelTranslationString={"CANCEL"}
/>

<div className="btm-spacer" />
</div>
)}
</Formik>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,6 @@ const DetailsMetadataTab = ({
isLast
/>

<div className="btm-spacer" />
</>
)}
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -427,8 +427,6 @@ const EditScheduledEventsEditPage = <T extends RequiredFormProps>({
}
}
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,6 @@ const EditScheduledEventsGeneralPage = <T extends RequiredFormProps>({
}
isFirst
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -522,8 +522,6 @@ const EventDetailsSchedulingTab = ({
cancelTranslationString="CANCEL"
isLast
/>

<div className="btm-spacer" />
</>
)}
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -360,8 +360,6 @@ const NewAccessPage = <T extends RequiredFormProps>({
}}
previousPage={previousPage}
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -133,8 +133,6 @@ const NewProcessingPage = <T extends RequiredFormProps>({
nextPage={nextPage}
previousPage={() => previous()}
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -225,8 +225,6 @@ const NewSourcePage = <T extends RequiredFormProps>({
}}
previousPage={previousPage}
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,6 @@ const SeriesDetailsThemeTab = ({
cancelTranslationString="CANCEL"
isLast
/>

<div className="btm-spacer" />
</>
)}
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -134,8 +134,6 @@ const StartTaskGeneralPage = <T extends RequiredFormProps>({
customValidation={!checkValidityStartTaskEventSelection(formik.values)}
isFirst
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,6 @@ const StartTaskWorkflowPage = <T extends RequiredFormProps>({
}}
customValidation={!(formik.values.workflow && formik.isValid)}
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
2 changes: 0 additions & 2 deletions src/components/events/partials/modals/DeleteEventsModal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,6 @@ const DeleteEventsModal = ({
nextTranslationString="WIZARD.DELETE"
previousTranslationString="CANCEL"
/>

<div className="btm-spacer" />
</>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -295,8 +295,6 @@ const EditMetadataEventsModal = ({
cancelTranslationString="CLOSE"
isLast
/>

<div className="btm-spacer" />
</>
)}
</Formik>
Expand Down
2 changes: 0 additions & 2 deletions src/components/users/partials/wizard/AclAccessPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -350,8 +350,6 @@ const AclAccessPage = <T extends RequiredFormProps>({
}
previousPage={previousPage}
/>

<div className="btm-spacer" />
</>
)}
</>
Expand Down
1 change: 0 additions & 1 deletion src/styles/components/_components-config.scss
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@


@use "simple-box";
@use "helper-classes";
@use "cal";
@use "ui";
@use "dropdowns";
Expand Down
39 changes: 0 additions & 39 deletions src/styles/components/_helper-classes.scss

This file was deleted.

2 changes: 1 addition & 1 deletion src/styles/main.scss
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ a {
top: 30px;
position: fixed;
max-height: 90vh;
overflow: scroll;
overflow: auto;
input[readonly] {
border: none;
}
Expand Down

0 comments on commit 75ddd63

Please sign in to comment.