github/SharpIDE
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

dispose of event subs

Browse Source
This commit is contained in:
Matt Parker
2025-10-27 18:53:37 +10:00
parent 7d1600df61
commit 4e93a7ae6d
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/SharpIDE.Godot/Features/CodeEditor/SharpIdeCodeEdit_SymbolHover.cs
View File

@@ -12,7 +12,6 @@ public partial class SharpIdeCodeEdit
private void CloseSymbolHoverWindow() private void CloseSymbolHoverWindow()
{ {
_symbolHoverTimer?.EmitSignal(Timer.SignalName.Timeout); _symbolHoverTimer?.EmitSignal(Timer.SignalName.Timeout);
_symbolHoverTimer = null;
} }
// This method is a bit of a disaster - we create an additional invisible Window, so that the tooltip window doesn't disappear while the mouse is over the hovered symbol // This method is a bit of a disaster - we create an additional invisible Window, so that the tooltip window doesn't disappear while the mouse is over the hovered symbol
@@ -82,13 +81,20 @@ public partial class SharpIdeCodeEdit
{ {
tooltipWindow.QueueFree(); tooltipWindow.QueueFree();
symbolNameHoverWindow.QueueFree(); symbolNameHoverWindow.QueueFree();
tooltipWindow.MouseExited -= StartTimer;
tooltipWindow.MouseEntered -= StopTimer;
symbolNameHoverWindow.MouseExited -= StartTimer;
symbolNameHoverWindow.MouseEntered -= StopTimer;
_symbolHoverTimer = null;
}; };
_symbolHoverTimer = timer; _symbolHoverTimer = timer;
tooltipWindow.MouseExited += () => timer.Start(); void StartTimer() => timer.Start();
tooltipWindow.MouseEntered += () => timer.Stop(); void StopTimer() => timer.Stop();
symbolNameHoverWindow.MouseExited += () => timer.Start(); tooltipWindow.MouseExited += StartTimer;
symbolNameHoverWindow.MouseEntered += () => timer.Stop(); tooltipWindow.MouseEntered += StopTimer;
symbolNameHoverWindow.MouseExited += StartTimer;
symbolNameHoverWindow.MouseEntered += StopTimer;
var styleBox = new StyleBoxFlat var styleBox = new StyleBoxFlat
{ {