Skip to content

[Due for payment 2026-05-20] [Due for payment 2026-05-20] [No QA] refactor: OptionRowLHN — DraftIndicator + PinIndicator subcomponents #89957

Description

@mountiny

Part of

#89946 — OptionRowLHN compound component decomposition

Step

11 of 12 — depends on #89947 (Provider + context interface). Incorporates work from #89754.

What

Two extractions in one PR:

  1. OptionRow.DraftIndicator (DraftIndicator.tsx) — pencil Icon; renders when hasDraftComment && optionItem.isAllowedToComment; reads from RowContext
  2. OptionRow.PinIndicator (PinIndicator.tsx) — pin Icon; renders when !brickRoadIndicator && optionItem.isPinned; reads from RowContext

Both carry their own accessibilityLabel (translated inside the component).

Why

Draft and pin indicators are mutually exclusive decoration icons that should each be self-contained. This carries forward the intent of #89754.

Acceptance criteria

  • Each indicator owns its render guard and accessibility label
  • Pencil and Pin icon logic removed from OptionRowLHN.tsx
  • No visible behavior or style changes
Issue OwnerCurrent Issue Owner: @DylanDylann

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions