Skip to content

Feature/windows icons#20

Merged
Louis-7 merged 6 commits intomainfrom
feature/windows-icons
Feb 12, 2026
Merged

Feature/windows icons#20
Louis-7 merged 6 commits intomainfrom
feature/windows-icons

Conversation

@Louis-7
Copy link
Contributor

@Louis-7 Louis-7 commented Feb 12, 2026

Pull Request Template

Description

  • Replace the default Electron icon with SnapMind icon.
  • The tray icon can now adapt to the system theme.

Related Issue

Closes #19

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor
  • Other

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates SnapMind’s Windows branding by improving tray icon handling (including theme-aware switching) and adjusting build configuration to better apply the app icon during Windows packaging.

Changes:

  • Added Windows tray icon theme adaptation via nativeTheme (switches between light/dark .ico assets on theme updates).
  • Updated Windows builder config to enable executable icon/resource editing.
  • Minor formatting/style cleanups across a few TS/TSX files and added jest-dom types to the TS config.

Reviewed changes

Copilot reviewed 6 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tsconfig.json Adds @testing-library/jest-dom types to compiler options.
src/services/AIService.ts Error throw formatting only.
src/components/ModelTable.tsx Fixes handler termination (;) and normalizes JSX attribute quoting/indentation.
src/components/Icon.tsx Normalizes quote style for react-icons/md import.
package-lock.json Lockfile updates (axios + follow-redirects versions).
main.ts Implements Windows tray icon selection + live updates on theme changes.
electron/assets/mind_tray_windows_white.ico New Windows tray icon variant for dark theme.
electron/assets/mind-white.svg Adds a white SVG asset (currently unused).
electron/SettingsService.ts Formatting only.
electron-builder-win.json Enables signAndEditExecutable to apply icon/resource edits during packaging.

@Louis-7 Louis-7 merged commit aa477a0 into main Feb 12, 2026
11 checks passed
@Louis-7 Louis-7 deleted the feature/windows-icons branch February 12, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The app icon on Windows is using the default Electron icon

2 participants