Closed m-mohr closed 4 years ago
Mainly agreed.
On screens with a lot of available height, using flex: 1
results in this:
But that can be fixed by using flex-shrink: 1
instead.
And you don't really need max-height: fit-content
, do you? (BTW it's -moz-fit-content
in Firefox, i.e. still experimental :confused:)
Seems like the fit-content
part didn't do anything in my Firefox (because it's experimental?), but relieved the too-big-box in your Chrome.
But flex-shrink
works in both, so we shall use that.
The functionalities box is scrollable although there's some space left on some screens. I'd propose to change the HTML/CSS a bit to support flex box stretching:
section#discover-filters
:display: flex; flex-direction: column
.endpointchooser.compact
replacemax-height: 5em;
withmax-height: fit-content; min-height: 3em; flex: 1
.