Refactor overlay components and improve tooltip functionality

- Removed the Windows timer overlay implementation as it is no longer needed.
- Simplified tooltip code by eliminating unused font size functions and streamlining the Show function.
- Added tests for image overlay title extraction and overlay bounds calculations.
- Introduced runtime text overlay with enhanced features including auto-close functionality and copy button.
- Implemented tests for runtime text overlay behaviors, including size adjustments and tooltip anchoring.
- Added tests for timer overlay size calculations to ensure proper expansion and text measurement.

Author: qianlifeng

wox.core/main.go

@@ -31,6 +31,7 @@ import (
 	"wox/util/clipboard"
 	"wox/util/imagecache"
 	"wox/util/mainthread"
+	"wox/util/overlay"
 	"wox/util/permission"
 	"wox/util/selection"
 
@@ -377,6 +378,7 @@ func run() {
 	}
 	coreServices := ui.NewCoreServices()
 	embeddedGoUIApp = golauncher.New(util.IsDev(), coreServices)
+	overlay.SetWindowManager(embeddedGoUIApp.WindowManager())
 	coreServices.AttachView(embeddedGoUIApp)
 	// Wire text-field copy/cut/paste to the cross-platform clipboard backend.
 	golauncher.SetClipboardProvider(golauncher.NewUtilClipboardProvider())

wox.core/plugin/system/ai_command.go

@@ -26,6 +26,7 @@ import (
 )
 
 var aiCommandIcon = common.PluginAICommandIcon
+var aiCommandTitleIcon, _ = aiCommandIcon.ToImage()
 
 var (
 	// Native overlays require the app main thread. Keeping the calls replaceable
@@ -373,37 +374,15 @@ func buildAICommandLoadingOptions(name string, position mouse.Point, message str
 			Anchor:           overlay.AnchorTopLeft,
 			OffsetX:          position.X + aiCommandLoadingOverlayOffsetX,
 			OffsetY:          position.Y + aiCommandLoadingOverlayOffsetY,
-			Width:            estimateAICommandLoadingOverlayWidth(message),
+			MinWidth:         aiCommandLoadingOverlayMinWidth,
+			MaxWidth:         aiCommandLoadingOverlayMaxWidth,
 		},
 		Message:  message,
 		Loading:  true,
-		FontSize: 12,
 		IconSize: aiCommandLoadingOverlayIconSize,
 	}
 }
 
-func estimateAICommandLoadingOverlayWidth(message string) float64 {
-	// Bug fix: the old fixed width was sized for "AI". Localized labels such as
-	// "Thinking..." need enough room after the spinner and padding are reserved,
-	// otherwise the native text view wraps into unreadable fragments.
-	textWidth := 0.0
-	for _, r := range message {
-		if r <= 0x7f {
-			textWidth += 7
-		} else {
-			textWidth += 12
-		}
-	}
-	width := 12 + float64(aiCommandLoadingOverlayIconSize) + 8 + textWidth + 12
-	if width < aiCommandLoadingOverlayMinWidth {
-		return aiCommandLoadingOverlayMinWidth
-	}
-	if width > aiCommandLoadingOverlayMaxWidth {
-		return aiCommandLoadingOverlayMaxWidth
-	}
-	return width
-}
-
 func (c *Plugin) showAICommandLoadingOverlay(ctx context.Context, name string) bool {
 	position, ok := mouse.CurrentPosition()
 	if !ok {
@@ -430,23 +409,26 @@ func (c *Plugin) currentAICommandOverlayPosition(ctx context.Context) mouse.Poin
 	return position
 }
 
-func (c *Plugin) showAICommandResultOverlay(ctx context.Context, name string, position *mouse.Point, streamResult common.ChatStreamData) {
+func (c *Plugin) showAICommandResultOverlay(ctx context.Context, name string, title string, position *mouse.Point, streamResult common.ChatStreamData) {
 	message := formatAICommandResultOverlayMessage(ctx, streamResult)
 	copyText := strings.TrimSpace(streamResult.Data)
 	opts := textoverlay.Options{
 		Window: overlay.WindowOptions{
 			ID:            name,
 			Topmost:       true,
+			TakeFocus:     true,
+			Width:         aiCommandResultOverlayMaxWidth,
 			MinWidth:      aiCommandResultOverlayMinWidth,
 			MaxWidth:      aiCommandResultOverlayMaxWidth,
 			MaxHeight:     aiCommandResultOverlayMaxHeight,
 			Movable:       true,
 			CloseOnEscape: true,
 		},
+		Title:        title,
+		TitleIcon:    aiCommandTitleIcon,
 		Closable:     true,
 		Message:      message,
 		Loading:      streamResult.Status == common.ChatStreamStatusStreaming && copyText == "",
-		FontSize:     12,
 		IconSize:     aiCommandLoadingOverlayIconSize,
 		FollowScroll: true,
 	}
@@ -649,7 +631,7 @@ func (c *Plugin) buildAICommandActions(ctx context.Context, command commandSetti
 				util.Go(ctx, "ai command run and show", func() {
 					overlayName := fmt.Sprintf("ai_command_run_and_show_result_%s", actionContext.ResultId)
 					position := c.currentAICommandOverlayPosition(ctx)
-					c.showAICommandResultOverlay(ctx, overlayName, &position, common.ChatStreamData{Status: common.ChatStreamStatusStreaming})
+					c.showAICommandResultOverlay(ctx, overlayName, command.Name, &position, common.ChatStreamData{Status: common.ChatStreamStatusStreaming})
 					lastOverlayUpdateAt := int64(0)
 					lastOverlayMessage := ""
 
@@ -664,7 +646,7 @@ func (c *Plugin) buildAICommandActions(ctx context.Context, command commandSetti
 								lastOverlayUpdateAt = now
 								lastOverlayMessage = message
 							}
-							c.showAICommandResultOverlay(ctx, overlayName, nil, streamResult)
+							c.showAICommandResultOverlay(ctx, overlayName, command.Name, nil, streamResult)
 						},
 					})
 					if final.Err != nil {

wox.core/plugin/system/dictation/dictation.go

@@ -1637,7 +1637,6 @@ func (p *DictationPlugin) showActionResultOverlay(ctx context.Context, text stri
 		Window:                   window,
 		Closable:                 true,
 		Message:                  text,
-		FontSize:                 14,
 		FollowScroll:             true,
 		ShowCopyButton:           true,
 		CopyButtonTooltip:        i18n.GetI18nManager().TranslateWox(ctx, "plugin_dictation_action_copy"),

wox.core/plugin/system/explorer/explorer.go

@@ -1106,10 +1106,8 @@ func (c *ExplorerPlugin) startOverlayListener(ctx context.Context) {
 				return
 			}
 			messageKey := "plugin_explorer_hint_message_dialog"
-			fontSize := 10.0
 			if util.IsMacOS() {
 				messageKey = "plugin_explorer_hint_message_dialog_macos"
-				fontSize = 12
 			}
 
 			title := window.GetWindowNameByPid(pid)
@@ -1125,8 +1123,7 @@ func (c *ExplorerPlugin) startOverlayListener(ctx context.Context) {
 					StickyWindowId:  dialogWindowId,
 					MaxWidth:        500,
 				},
-				Message:  c.api.GetTranslation(localCtx, messageKey),
-				FontSize: fontSize,
+				Message: c.api.GetTranslation(localCtx, messageKey),
 				OnClick: func() bool {
 					clickCtx := context.WithValue(ctx, util.ContextKeyTraceId, uuid.NewString())
 					clickCtx = util.WithCoreSessionContext(clickCtx)

wox.core/plugin/system/screenshot.go

@@ -843,7 +843,6 @@ func (p *ScreenshotPlugin) pinScreenshotToScreen(ctx context.Context, screenshot
 		Width:         width,
 		Height:        height,
 		Movable:       true,
-		Shadow:        true,
 		CloseOnEscape: true,
 		// Bug fix: Windows native overlays normally position screen overlays relative to the
 		// primary work area. Screenshot selections are already desktop-absolute, so pinning must

wox.core/plugin/system/sys/sys.go

@@ -479,11 +479,10 @@ func (r *SysPlugin) buildDevCommands() []SysCommand {
 			Icon:  common.CPUProfileIcon,
 			Action: func(ctx context.Context, actionContext plugin.ActionContext) {
 				img, _ := common.WoxIcon.ToImage()
-				notifier.Notify(img, `This is a very long notification message to test the notification system in Wox.
-				If you see this message, the notification system is working properly.
-				You can customize the duration, appearance, and behavior of notifications as needed.
-				Enjoy using Wox!
-				`+time.Now().String())
+				notifier.Notify(img, "This is a very long notification message to test the notification system in Wox.\n"+
+					"If you see this message, the notification system is working properly.\n"+
+					"You can customize the duration, appearance, and behavior of notifications as needed.\n"+
+					"Enjoy using Wox!\n"+time.Now().String())
 			},
 		},
 

wox.core/plugin/system/timer.go

@@ -567,9 +567,9 @@ func (t *TimerPlugin) tick(ctx context.Context) {
 			toRemove = append(toRemove, resultID)
 			return true
 		}
-		if !t.pushTimerResultUpdate(ctx, resultID, timerID) {
-			toRemove = append(toRemove, resultID)
-		}
+		// The first tick can race with the query result reaching the UI. Keep the
+		// active timer tracked so a transient miss is retried on the next tick.
+		t.pushTimerResultUpdate(ctx, resultID, timerID)
 		return true
 	})
 	for _, resultID := range toRemove {

wox.core/plugin/system/timer_test.go

@@ -4,8 +4,24 @@ import (
 	"context"
 	"testing"
 	"time"
+	"wox/plugin"
+	"wox/util"
 )
 
+type timerUpdateTestAPI struct {
+	plugin.API
+	updates int
+}
+
+func (a *timerUpdateTestAPI) IsVisible(context.Context) bool {
+	return true
+}
+
+func (a *timerUpdateTestAPI) UpdateResult(context.Context, plugin.UpdatableResult) bool {
+	a.updates++
+	return a.updates > 1
+}
+
 func TestParseTimerQuery(t *testing.T) {
 	tests := []struct {
 		input     string
@@ -154,3 +170,23 @@ func TestTimerPersistedRoundTrip(t *testing.T) {
 		t.Fatalf("deadline=%v want %v", restored.Deadline, original.Deadline)
 	}
 }
+
+func TestTimerTickRetriesTransientResultMiss(t *testing.T) {
+	api := &timerUpdateTestAPI{}
+	timer := &TimerPlugin{
+		api:            api,
+		timers:         map[string]*timerEntry{"t1": {ID: "t1", DurationLabel: "1m", Deadline: time.Now().Add(time.Minute)}},
+		trackedResults: util.NewHashMap[string, string](),
+	}
+	timer.trackedResults.Store("t1", "t1")
+
+	timer.tick(context.Background())
+	timer.tick(context.Background())
+
+	if api.updates != 2 {
+		t.Fatalf("updates=%d want 2", api.updates)
+	}
+	if _, ok := timer.trackedResults.Load("t1"); !ok {
+		t.Fatal("active timer should remain tracked after a transient update miss")
+	}
+}

wox.core/plugin/system/window_manager/window_groups.go

@@ -31,7 +31,6 @@ const (
 const (
 	windowGroupLaunchPlaceholderPrefix       = "window_group_launch_"
 	windowGroupLaunchPlaceholderCornerRadius = 8.0
-	windowGroupLaunchPlaceholderFontSize     = 18.0
 )
 
 const (
@@ -469,7 +468,6 @@ func (p *WindowManagerPlugin) showWindowGroupLaunchPlaceholder(ctx context.Conte
 		Message:       message,
 		Loading:       true,
 		CenterContent: true,
-		FontSize:      windowGroupLaunchPlaceholderFontSize,
 	})
 	p.api.Log(ctx, plugin.LogLevelInfo, fmt.Sprintf("window manager showed launch placeholder: group=%s app=%s identity=%s overlay=%s rect=%+v", group.Id, placement.AppName, placement.Identity, name, placement.Rect))
 	return name

wox.core/ui/launcher/app.go

@@ -227,6 +227,11 @@ func New(isDev bool, services contract.Services) *App {
 	return app
 }
 
+// WindowManager returns the process-local registry shared by launcher-owned secondary surfaces.
+func (a *App) WindowManager() *woxui.WindowManager {
+	return a.windows
+}
+
 // newApp builds isolated launcher state while sharing only process-wide window and message infrastructure.
 func newApp(isDev bool, services contract.Services, windows *woxui.WindowManager, instances *appInstanceRegistry, primary *App, isPrimary bool, instanceName string, windowID woxui.WindowID) *App {
 	sessionID := newID()