IntelliSense icon indicating copy to clipboard operation
IntelliSense copied to clipboard

Tooltip displayed on wrong screen

Open govert opened this issue 3 years ago • 9 comments

However, there is a new weird bug that wasn't there in the previous version. This affects computers with more than one screen. The bug appears in both Excel 2010 and Excel 365. Tested on Windows 10. Only the user functions are affected, but built in Excel functions are fine.

Bug: When you have 2 screens, and make the RIGHT screen the main screen, then when the Excel window is in the left screen, the tooltip gets placed on the other screen rather than next to the cell.

To reproduce: Set your desktop to have two screens. Set the RIGHT screen as the main screen, and put excel into the left screen. When the LEFT screen is the main screen, seems to work fine.

image

Originally posted by @Nik-MO in https://github.com/Excel-DNA/IntelliSense/issues/104#issuecomment-925261515

govert avatar May 23 '22 21:05 govert

I have replicated this issue here as well. Has there been any progress since you opened the issue?

gmichaud avatar Sep 29 '22 13:09 gmichaud

Hi @gmichaud - No sorry, I've not had a look yet - I'll bump up a bit.

govert avatar Sep 29 '22 19:09 govert

@gmichaud I've tried to tweak the location code a bit in the new version 1.6.1-alpha2. The calculations and position tracking are a bit tricky, so I'm hoping nothing has broken. But it seems to work now in the reported problem scenario.

govert avatar Oct 09 '22 22:10 govert

@govert we cherry-picked your commit to our fork, but the fix does not seem to be working for us.

sshukurov avatar Oct 10 '22 12:10 sshukurov

Debug output: image

sshukurov avatar Oct 10 '22 13:10 sshukurov

@sshukurov Can you try with a simple add-in with the NuGet packages 1.6.1-alpha2, and make sure it is the only add-in being loaded into the Excel process when you test?

govert avatar Oct 10 '22 13:10 govert

@govert it seems to work correctly when I create an add-in from scratch and reference the relevant pre-release NuGet packages directly.

sshukurov avatar Oct 10 '22 14:10 sshukurov

@sshukurov I forgot to update the IntelliSenseServer version before pushing the package. That might cause an older version add-in with IntelliSense to remain loaded, if loaded in the same process. But then I can't explain why you'd get the debug output from your add-in. You might want to pull the last commit with the server version update too.

govert avatar Oct 10 '22 14:10 govert

@govert I just found our fork contains some additional lines of our own in ToolTipForm.cs - I commented out those and it helped!

Thanks a lot for helping with this.

sshukurov avatar Oct 10 '22 18:10 sshukurov