change the .nav-category height to min-height

[tangollama]

It's currently set to 254px on desktop at least. Some of the elements are larger, so I think that needs to be...

.nav-category .category-links {
min-height: 254px;
}

I'd make the change myself, but I don't want to mess with your mojo. If you want me to do it, assign back to me.

cc: @jkleinsc

[jglovier]

Yeah I struggled a bit with how to work this out when I was mocking it up, because a min-height will break the flow of the cards then, as so:

I ended up settling on using overflow: scroll, but the challenge then is that you don't know there are more options unless you actually scroll.

I'm going to try out using overflow: scroll again as a fix, and set the height so that it will at least show some of one of the next buttons. That will provide a visual indicator that there are more options. We can always iterate on the visual indicator later, and if it turns out that hiding some options at all behind a scroll behavior is a bad idea I can rethink this approach later.

So basically FYI I'm punting on the UX of this piece for now by implementing the scroll thing.

[jglovier]

Cards with additional nav items beyond view look like this for now: