Replace outdated PNG button icons with Fluent glyphs#138
Merged
Conversation
The existing PNG chrome icons look out of place in the Windows 11 Fluent UI — they're orange, glossy, and faux-3D in a style that predates the rest of the app's aesthetic. Swapping them for `FluentIcons.Wpf` vector glyphs brings the per-row icons and action buttons in line with everything else. - Added `GlyphButton` (drop-in replacement for `PictureButton`) hosting a `FluentIcon` bound to an `Icon` dependency property. - Added `AppGlyphs` — a catalog mapping each former PNG (`Add`, `Up`, `Down`, `Delete`, `Setting`, …) to a Fluent `Icon` constant. - `MyListItem` now has a `Glyph` property alongside `Image`; setting one clears the other. The glyph's `Foreground` is bound to the title `TextBlock`, so row icons inherit the list's foreground color and follow the current theme. - Migrated every `PictureButton(AppImage.*)` call site and every `Image = AppImage.*` list-row assignment across `FoldSubItem`, `NewItem`, `RestoreItem`, `SendToList`, `ShellItem`, `ShellList`, `ShellSubMenuDialog`, `SwitchDicList`, `WinXList`, and the `IBtn*` / `ITsiGuidItem` interface helpers. - Deleted the 20 now-unused PNGs, the `AppImage`/`AppResources` entries that referenced them, and `PictureButton.cs`. - Added a `FluentIcons.Wpf` 2.1.324 package reference.
Jack251970
approved these changes
Apr 23, 2026
Owner
Jack251970
left a comment
Jack251970
left a comment
There was a problem hiding this comment.
Thanks for your contribution!
3 tasks
Jack251970
pushed a commit
that referenced
this pull request
Apr 24, 2026
The app icon and README banner were holdovers from an earlier era -- a glossy, faux-3D orange mouse that reads as Windows 7. Since the rest of the UI has moved to Fluent (and #138 just swapped the per-row PNG chrome for Fluent glyphs), the icon was the last piece that still looked dated. This change redesigns both the app icon and the README banner in a flatter, Fluent-aligned style, and commits the SVG sources alongside the rasters so future tweaks don't need a round trip to a designer. - Redesigns the app icon (AppIcon.ico + Properties/Resources/Images/Logo.png) and the README header banner (Logo/Logo.png). - Commits the SVG masters: Properties/AppIcon.svg, Logo/Logo.svg (Chinese), Logo/Logo-en.svg (English). - AppIcon.ico now embeds sizes 16/20/24/32/40/48/64/96/128/256 so the icon stays sharp across Explorer, taskbar, title bar, and Alt+Tab at any DPI. - Adds an English banner (Logo/Logo-en.png); README-en.md used to point at the Chinese banner and now references the English one. Both banners are rasterized at the same per-unit density and padded to a shared canvas so <img width="700"> renders them at identical scale in either README. - Fixes src="/Logo/Logo.png" in both READMEs to ./Logo/... . The leading slash renders on GitHub but breaks in VS Code's markdown preview, which resolves it against the workspace root. - Adds generate-icons.ps1 (PowerShell 7 + resvg + magick) to reproduce every raster from the SVGs. - Drops the icon-attribution lines from both READMEs since that icon is being replaced.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The existing PNG chrome icons (new-item
+, delete trash, settings gear, fold chevrons, etc.) look out of place in the Windows 11 Fluent UI — they're orange, glossy, and faux-3D in a style that predates the rest of the app's aesthetic. This PR swaps them forFluentIcons.Wpfvector glyphs so the per-row icons and action buttons match everything else, and they now inherit the theme's foreground color automatically.Changes
GlyphButton(swap-in replacement forPictureButton) hosting aFluentIconbound to anIcondependency property.AppGlyphs— a catalog mapping each former PNG name (Add,Up,Down,Delete,Setting, …) to a FluentIconconstant.MyListItemnow exposes aGlyphproperty alongsideImage. Setting one clears the other, and the glyph'sForegroundis bound to the titleTextBlock, so row icons pick up the list's foreground color.PictureButton(AppImage.*)call site and everyImage = AppImage.*list-row assignment acrossFoldSubItem,NewItem,RestoreItem,SendToList,ShellItem,ShellList,ShellSubMenuDialog,SwitchDicList,WinXList, and theIBtn*/ITsiGuidItemhelper buttons.AppImage/AppResourcesentries that referenced them, andPictureButton.cs.FluentIcons.Wpf2.1.324 package reference.Screenshots
Before
After