Commit e8d1f972 authored by Kyle Keating's avatar Kyle Keating Committed by Kyle Keating

Limit length of the user filter list in notebook

This prevents long lists from going off-screen and becoming unwieldy.
parent 624b29e6
...@@ -43,7 +43,7 @@ function FilterSelect({ ...@@ -43,7 +43,7 @@ function FilterSelect({
); );
return ( return (
<Menu label={menuLabel} title={title}> <Menu label={menuLabel} title={title} contentClass="FilterSelect__menu">
{filterOptions.map(filterOption => ( {filterOptions.map(filterOption => (
<MenuItem <MenuItem
onClick={() => onSelect(filterOption)} onClick={() => onSelect(filterOption)}
......
...@@ -2,6 +2,17 @@ ...@@ -2,6 +2,17 @@
@use "../../variables" as var; @use "../../variables" as var;
.FilterSelect { .FilterSelect {
&__menu {
// Prevent the list from overflowing off screen in most cases.
max-height: 70vh;
overflow-y: auto;
// Limit the length for large screens.
@media only screen and (min-height: 44em) {
max-height: 36em;
}
}
&__menu-label { &__menu-label {
@include utils.font--large; @include utils.font--large;
align-items: center; align-items: center;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment