Skip to content

Commit e66951c

Browse files
Muhammad Faraz  MaqsoodMuhammad Faraz  Maqsood
authored andcommitted
fix: height for lti modal
1 parent b54dff3 commit e66951c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/index.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -367,20 +367,20 @@
367367
// window (retaining padding around the edge). Bootstrap modals don't have a full-screen
368368
// size like this. Because of the hack below around react-focus-on's div, it would be better long-term to pull this into Paragon and perhaps call it "modal-full" or something like that.
369369
.modal-lti {
370-
height: 100%;
370+
height: 80vh;
371371
max-width: 100% !important;
372372

373373
// I don't like this. We need to set a height of 100% on a div created by react-focus-on, a
374374
// package we use in our Modal. That div has no class name or ID, so instead we're uniquely
375375
// identifying it by based on a unique attribute it has which its siblings don't share.
376376
> div[data-focus-lock-disabled="false"] {
377-
height: 100%;
377+
height: 80vh;
378378
}
379379

380380
// Along with setting the height of modal-content's parent div from react-focus-on, we need to
381381
// set modal-content's height as well to get the modal to expand to full-screen height.
382382
.modal-content {
383-
height: 100%;
383+
height: 80vh;
384384
}
385385
}
386386

0 commit comments

Comments
 (0)