fix: Prevent IME tooltips from being deleted by parent window cleanup

Author: akiyosi

editor/imetooltip.go

@@ -147,6 +147,15 @@ func (i *IMETooltip) show() {
 		return
 	}
 
+	i.Show()
+	i.isShown = true
+}
+
+func (i *IMETooltip) setGrid() {
+	if i.s == nil || i.s.ws == nil {
+		return
+	}
+
 	if !(i.s.ws.palette != nil && i.s.ws.palette.widget != nil && i.s.ws.palette.widget.IsVisible()) {
 		win, ok := i.s.getWindow(i.s.ws.cursor.gridid)
 		if !ok || win == nil {
@@ -163,9 +172,7 @@ func (i *IMETooltip) show() {
 		i.setFont(i.s.font)
 	}
 
-	i.Show()
 	i.Raise()
-	i.isShown = true
 }
 
 func (i *IMETooltip) updateVirtualCursorPos() {

editor/screen.go

@@ -1062,6 +1062,7 @@ func (s *Screen) gridCursorGoto(args []interface{}) {
 			}
 
 			win.raise()
+			win.s.tooltip.setGrid()
 
 			// reset smooth scroll scrolling offset
 			win.scrollPixels2 = 0
@@ -1508,7 +1509,7 @@ func (s *Screen) update() {
 	s.windows.Range(func(grid, winITF interface{}) bool {
 		win := winITF.(*Window)
 		// if grid is dirty, we remove this grid
-		if win.isGridDirty {
+		if win.isGridDirty && !win.isSameAsCursorGrid() {
 			// if win.queueRedrawArea[2] > 0 || win.queueRedrawArea[3] > 0 {
 			// 	// If grid has an update area even though it has a dirty flag,
 			// 	// it will still not be removed as a valid grid

editor/window.go

@@ -1169,7 +1169,7 @@ func (w *Window) applyTemporaryMousescroll(ms string) {
 	}
 }
 
-func (w *Window) isEventEmitOnCursorGrid() bool {
+func (w *Window) isSameAsCursorGrid() bool {
 	return w.grid == w.s.ws.cursor.gridid
 }
 

editor/workspace.go

@@ -2462,6 +2462,7 @@ func (ws *Workspace) InputMethodEvent(event *gui.QInputMethodEvent) {
 			ws.screen.tooltip.hide()
 			ws.screen.refresh()
 		} else {
+			ws.screen.tooltip.setGrid()
 			ws.screen.tooltip.show()
 			ws.screen.tooltip.parsePreeditString(preeditString)
 			ws.screen.tooltip.update()