maximus/Simpl-Resultat

Fork 0

feat(branding): replace default Tauri icon with custom design #186

Closed

maximus wants to merge 0 commits from feat/app-icon into main

maximus commented

2026-05-02 18:52:56 +00:00

Owner

Résumé

Nouvelle icône d'application : calculatrice à visage de robot souriant avec un cadenas de confidentialité sur la touche Entrée / =. Reflète les 4 valeurs du produit — robot (assistant), simplicité (formes géométriques), comptabilité (calculatrice), confidentialité (cadenas).

Détails

Source SVG conservée à src-tauri/icons/icon.svg + public/icon.svg (favicon web)
16 fichiers raster spécifiques aux plateformes régénérés via tauri icon icon.svg
Suppression des SVG par défaut Vite/Tauri inutilisés (public/vite.svg, public/tauri.svg)
<title> corrigé : « Tauri + React + Typescript » → « Simpl'Résultat »
.gitignore : src-tauri/icons/{ios,android}/ (hors-scope, targets desktop seulement : nsis/deb/rpm)

Genèse

Design produit dans le spike ~/claude-code/.spikes/logo/ (4 concepts SVG explorés, Claude direct sans coût IA). Choix retenu : variante 04b (calculatrice + cadenas).

Test plan

CI verte (check.yml : cargo check + cargo test + npm build + npm test)
Visual : ouvrir l'app en local (npm run tauri dev), vérifier l'icône taskbar/dock
Visual : favicon web visible dans l'onglet navigateur en mode dev
Build NSIS : icône correcte sur Windows
Build .deb/.rpm : icône correcte sur Linux

## Résumé Nouvelle icône d'application : calculatrice à visage de robot souriant avec un cadenas de confidentialité sur la touche Entrée / `=`. Reflète les 4 valeurs du produit — robot (assistant), simplicité (formes géométriques), comptabilité (calculatrice), confidentialité (cadenas). ## Détails - Source SVG conservée à `src-tauri/icons/icon.svg` + `public/icon.svg` (favicon web) - 16 fichiers raster spécifiques aux plateformes régénérés via `tauri icon icon.svg` - Suppression des SVG par défaut Vite/Tauri inutilisés (`public/vite.svg`, `public/tauri.svg`) - `<title>` corrigé : *« Tauri + React + Typescript »* → *« Simpl'Résultat »* - `.gitignore` : `src-tauri/icons/{ios,android}/` (hors-scope, targets desktop seulement : nsis/deb/rpm) ## Genèse Design produit dans le spike `~/claude-code/.spikes/logo/` (4 concepts SVG explorés, Claude direct sans coût IA). Choix retenu : variante 04b (calculatrice + cadenas). ## Test plan - [ ] CI verte (`check.yml` : cargo check + cargo test + npm build + npm test) - [ ] Visual : ouvrir l'app en local (`npm run tauri dev`), vérifier l'icône taskbar/dock - [ ] Visual : favicon web visible dans l'onglet navigateur en mode dev - [ ] Build NSIS : icône correcte sur Windows - [ ] Build .deb/.rpm : icône correcte sur Linux

maximus added 1 commit 2026-05-02 18:52:57 +00:00

feat(branding): replace default Tauri icon with custom design

PR Check / rust (pull_request) Successful in 22m51s

Details

PR Check / frontend (pull_request) Successful in 2m24s

Details

d147520d6b

Robot-faced calculator with a privacy lock on the Enter / `=` key.
Conveys the four product values: robot (assistant), simplicity
(geometric shapes), accounting (calculator), privacy (lock).

- New source SVG at src-tauri/icons/icon.svg (kept in repo for future
  iterations) and public/icon.svg (web favicon)
- Regenerated 16 platform-specific raster icons via `tauri icon`
- Removed unused default Vite/Tauri SVG assets from public/
- Fixed window <title> ("Tauri + React + Typescript" → "Simpl'Résultat")
- gitignore ios/ and android/ subdirs (out-of-scope, desktop-only targets)

maximus added the

source:human

status:review

type:feature

labels 2026-05-02 18:53:09 +00:00

maximus commented

2026-05-02 19:00:09 +00:00

Author

Owner

Verdict : APPROVE

Summary : Remplacement propre de l'icone Tauri par defaut. Diff self-contained (assets + index.html + .gitignore + CHANGELOG bilingue). Pas de risque securite, pas de logique applicative touchee.

Verifications

Tous les rasters declares dans src-tauri/tauri.conf.json (32x32, 128x128, 128x128@2x, icon.icns, icon.ico) sont regeneres et presents sur la branche.
index.html : favicon /icon.svg + <title>Simpl'Resultat</title>. Aucune reference orpheline a vite.svg / tauri.svg dans src/, public/ ou index.html.
SVG source double : src-tauri/icons/icon.svg (iteration future) et public/icon.svg (favicon web). Meme blob hash, coherent.
.gitignore : src-tauri/icons/{ios,android}/ couvre proprement les sous-dossiers generes en bonus par tauri icon (coherent avec targets desktop-only nsis/deb/rpm).
CHANGELOG : entree bilingue FR/EN sous [Unreleased] / [Non publie], categorie Changed/Modifie.
Commit conventionnel : feat(branding): ....

Suggestions (non bloquantes)

64x64.png orphelin : ajoute dans la PR mais non liste dans bundle.icon de tauri.conf.json. Soit l'ajouter a la liste pour qu'il soit bundle, soit le retirer pour eviter du dead asset (~1.8 ko, broutille).
Pas de Fixes #N dans le body. Non bloquant ici. Si une issue de tracking existe (branding/scaffolding cleanup), la lier pour fermeture auto au merge.
Test plan visuel : 4 cases a cocher manuellement (taskbar/dock/favicon/builds NSIS/deb/rpm). A valider avant tag de release pour eviter une icone cassee en prod.

Code reviewed by Claude Opus 4.7 (1M context)

## Verdict : APPROVE **Summary** : Remplacement propre de l'icone Tauri par defaut. Diff self-contained (assets + `index.html` + `.gitignore` + CHANGELOG bilingue). Pas de risque securite, pas de logique applicative touchee. ### Verifications - Tous les rasters declares dans `src-tauri/tauri.conf.json` (`32x32`, `128x128`, `128x128@2x`, `icon.icns`, `icon.ico`) sont regeneres et presents sur la branche. - `index.html` : favicon `/icon.svg` + `<title>Simpl'Resultat</title>`. Aucune reference orpheline a `vite.svg` / `tauri.svg` dans `src/`, `public/` ou `index.html`. - SVG source double : `src-tauri/icons/icon.svg` (iteration future) et `public/icon.svg` (favicon web). Meme blob hash, coherent. - `.gitignore` : `src-tauri/icons/{ios,android}/` couvre proprement les sous-dossiers generes en bonus par `tauri icon` (coherent avec targets desktop-only nsis/deb/rpm). - CHANGELOG : entree bilingue FR/EN sous `[Unreleased]` / `[Non publie]`, categorie `Changed`/`Modifie`. - Commit conventionnel : `feat(branding): ...`. ### Suggestions (non bloquantes) 1. **`64x64.png` orphelin** : ajoute dans la PR mais non liste dans `bundle.icon` de `tauri.conf.json`. Soit l'ajouter a la liste pour qu'il soit bundle, soit le retirer pour eviter du dead asset (~1.8 ko, broutille). 2. **Pas de `Fixes #N`** dans le body. Non bloquant ici. Si une issue de tracking existe (branding/scaffolding cleanup), la lier pour fermeture auto au merge. 3. **Test plan visuel** : 4 cases a cocher manuellement (taskbar/dock/favicon/builds NSIS/deb/rpm). A valider avant tag de release pour eviter une icone cassee en prod. --- Code reviewed by Claude Opus 4.7 (1M context)

maximus added

status:approved

and removed

status:review

labels 2026-05-02 19:00:14 +00:00

maximus referenced this pull request from a commit

2026-05-02 19:01:37 +00:00

feat(branding): bundle 64x64 icon in tauri.conf

maximus added 1 commit 2026-05-02 19:01:38 +00:00

feat(branding): bundle 64x64 icon in tauri.conf

PR Check / rust (pull_request) Successful in 22m29s

Details

PR Check / frontend (pull_request) Successful in 2m23s

Details

8030a4a1c4

Follow-up to PR #186 review: tauri icon CLI generates 64x64.png
but it was not declared in bundle.icon, so packagers (deb/rpm)
weren't picking it up. Add it alongside the other Linux sizes.

maximus closed this pull request

2026-05-02 19:46:05 +00:00

maximus commented

2026-05-02 19:46:05 +00:00

Author

Owner

Merge fait localement sur main (commit chain). Conflit CHANGELOG.md / CHANGELOG.fr.md résolu en gardant les deux entrées sous Changed/Modifié (icône + onboarding card).

maximus referenced this pull request from a commit

2026-05-02 19:46:06 +00:00

Merge PR #186: feat(branding) replace default Tauri icon + bundle 64x64

PR Check / rust (pull_request) Successful in 22m29s

Details

PR Check / frontend (pull_request) Successful in 2m23s

Details