search

icecoder / ICEcoder

Browser code editor awesomeness
http://icecoder.net
Other
1.41k stars 348 forks source link

Tooltip (Code Assist?) taking over entire editor screen #968

Closed raymondsalas closed 3 years ago

raymondsalas commented 3 years ago

There are instances where there are tooltips taking over my code editor screen. See screenshot below. Sometimes, it completely covers the editor screen that I cannot hover to anything else in the screen to remove the tooltip. Not sure if this is code assist related since I disabled the feature and it is still persisting.

In the screenshot, I have sweetalert2 (sweetalert2.github.io) loaded in my root directory. Which are the codes being displayed in the screenshot. This occurs when I am editing a non-sweetalert JS file and hover to the "document" word of "document.getElementById(elem)"

I was only able to disable the tooltip from appearing by adding: get('tooltip').style.display = "hidden";

under the if statement (icecoder.js > functionArgsTooltip): if (1 === numResults && -1 === [null, "def"].indexOf(cM.getTokenTypeAt(coordsChar)))

image

mattpass commented 3 years ago

Looks like I need to handle excess tooltip data better. Should be an easy fix, will sort that.

raymondsalas commented 3 years ago

It seems that terminal results may also need tweaking or word-wrapping if the line result is greater than code editor size.

image

If I use, shift+mouse wheel down, other text are exposed. But the black background is cut-off, making other text harder to read:

image

mattpass commented 3 years ago

OK, both fixed - CSS in terminal needed to wrap and break as you'd expect and tooltip now limited to 200 char max re functions: https://github.com/icecoder/ICEcoder/commit/78e417add4ce575f3ff7cf5d5e6c3377b2f8a205

Thanks for reporting both 👍🏼