Some restyling

[simurai]

This restyles the tooltip a bit:

Note: I haven't installed Go, so the text is just some dummy text.

[simurai]

I tried to add some scrolling, but also didn't get it to work. I guess the editor disables scrolling on those overlays. Because it would be scrolling inside a scrolling area and maybe annoying.

How much content will there usually be?

[zmb3]

The amount of content totally depends on what the user gets doc for. Generally pretty short for a function, but for a package it could be quite long.

I know what the problem is here, I just don't know how to fix it.

Right now, the size of the div that gets created is always big enough to fit the text, so it doesn't display scrollbars because it doesn't think they're needed. Even if there's a ton of text, it just makes the div thousands of pixels tall (even though most of it is off screen). If I set max-height to something small I do get scrollbars.

This isn't a great solution - it requires me to pick some arbitrary height, and it still behaves funky if the tooltip is displayed near the bottom of the editor.

[simurai]

Yeah, ideally it would try to figure out how much space is left at the bottom, if there isn't much, it would show the overlay above. And maybe even taking scroll position into account. I think autocomplete-plus tries to do that, but that's probably custom made.

[simurai]

btw. this is good timing, because if atom/atom#11875 gets merged, the .close button would be removed. We could keep it in case you think it's useful.

Or another option might be to auto-close the tooltip as soon as you move the cursor away. Or click somewhere outside. Then the close X wouldn't be needed.

[zmb3]

Thanks so much for this!

I do like the close button. I think closing the tooltip if the mouse is moved away can be jarring. I generally close the tooltip with the esc key, but for new users that's not always easy or intuitive to discover.

[simurai]

ok, I'll make a note about adding the .close button to atom/atom#11875.