No scrollbar when the dropdown for dashboards is taller than the vertical resolution

[SEJeff]

I used the excellent giraffe-collectd generator to autocreate a dashboards.js file based on all of the collectd hosts that I have in graphite.

The problem is that I've got thousands of hosts in graphite, so the dropdown where it says "Dashboards" is longer than the vertical resolution. Upstream bootstrap doesn't give a crap.

So the solution is simply adding this to bootstrap.js under .dropdown-menu

   max-height: 500px;
   overflow-y: auto;
   overflow-x: hidden;

You could hack this into the index.html via a style attribute as well on the dropdown-menu div, or you could move to using something like select2, which would give you a nice find as you type search of all of the menus. Obviously I can do this quite easily on my own "version" of giraffe, but I'd like to see changes incorporated upstream if possible. What do you think?

[gingerlime]

Hey @SEJeff - sorry, I somehow missed this issue.

On the face of it, it looks good and I'd be happy to merge this change in, but want to double-check that it doesn't break anything. Something like select2 sounds like a good idea too, and I'll look into it as well at some point.

@nhapke - what's your opinion on this?

[SEJeff]

Thanks!