microsoft/Terminal

Refactor IconPathConverter::IconWUX() to not set icon size

Open

#19806 opened on Jan 28, 2026

View on GitHub
 (3 comments) (0 reactions) (0 assignees)C++ (35,764 stars) (3,212 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Terminal

Description

I hit this problem pretty regularly whenever I'm working with icons (especially in the settings UI).

IconPathConverter::IconWUX() is a really powerful function that we use throughout the codebase to convert an icon path/code into an actual icon UI. However, we explicitly set the size of the icon on the way out. This was great when it was introduced because it meant that the icons in the dropdown, tabs, etc. all had the correct size. But as the project has grown to use those icons elsewhere (namely in the Settings UI), this has become a huge pain. Icons aren't always drawn to be the same size, especially if they're coming from different sources (i.e. hardcoded font icon vs evaluated icon in profile).

I'd update it whenever I hit the problem, but it's realistically a fundamental enough change that it will have side effects. I'd rather have it be its own change and audit the app to make sure it doesn't break anything.

References

Contributor guide

Tech stack
cpp
Domain
desktop
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Familiarity with C++Understanding of Windows Terminal settings UI
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
Start by examining the IconPathConverter::IconWUX() function in the referenced file (src/cascadia/UIHelpers/IconPathConverter.cpp line 318). Understand how it currently sets icon size and how it is called throughout the codebase, particularly in the settings editor (src/cascadia/TerminalSettingsEditor). Then, after removing the size setting, test all icon usages to ensure they render correctly without explicit sizing, and adjust any callers that rely on the old behavior.