Skip to content

[M3] Migrate InlineIcon usages #95159

Description

@mountiny

Problem

Several buttons render icons inline within text via Icon's inline prop, keeping inline layout logic inside the monolithic Icon facade. After M1 lands the dedicated InlineIcon primitive, these call sites should import it directly so the inline branch can be removed from Icon.

Solution

Replace each Icon usage that sets inline with a direct InlineIcon import, mapping props to the primitive's API (contentSize, iconWidth, iconHeight, etc.). Once no call sites remain, remove the inline prop and its branch from src/components/Icon/index.tsx.

Known call sites on main:

Verify for any additional Icon + inline usages after M1 merges (the M1 PR cites three call sites).

Acceptance criteria

  • No remaining Icon usages with inline in src/.
  • Inline copy and icon alignment unchanged in affected buttons.

Dependencies

Blocked by #95156 (M1 must merge first).

Issue OwnerCurrent Issue Owner: @jmusial

Metadata

Metadata

Labels

InternalRequires API changes or must be handled by Expensify staffReviewingHas a PR in reviewTaskWeeklyKSv2

Type

No type

Projects

Status
SUBISSUE

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions