Skip to content

fix(ui): align the transaction detail sheet with the app's sheet conv… - #58

Merged
Renekakpo merged 1 commit into
developfrom
fix/transaction-detail-design-system
Aug 12, 2026
Merged

fix(ui): align the transaction detail sheet with the app's sheet conv…#58
Renekakpo merged 1 commit into
developfrom
fix/transaction-detail-design-system

Conversation

@Renekakpo

Copy link
Copy Markdown
Contributor

…entions

Why:

  • The sheet was the only one with an icon at header right where every sibling puts text, and the only one whose hero glyph was a circle while the identical category glyph is a rounded square in the list rows. The delete control rendered as an empty Text to screen readers, naming the action for nobody, and the title was hardcoded English.

What:

  • Delete reads as text using the existing common.delete label, the hero glyph takes the same radius as the list rows, both header controls carry a proper accessibility role, and the title is translated across the five complete locales. The list renderers are untouched.

…entions

Why:
- The sheet was the only one with an icon at header right where every sibling
  puts text, and the only one whose hero glyph was a circle while the identical
  category glyph is a rounded square in the list rows. The delete control
  rendered as an empty Text to screen readers, naming the action for nobody, and
  the title was hardcoded English.

What:
- Delete reads as text using the existing common.delete label, the hero glyph
  takes the same radius as the list rows, both header controls carry a proper
  accessibility role, and the title is translated across the five complete
  locales. The list renderers are untouched.
@Renekakpo
Renekakpo merged commit efbfd33 into develop Aug 12, 2026
1 check passed
@Renekakpo
Renekakpo deleted the fix/transaction-detail-design-system branch August 12, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant