Skip to content

Improve transaction detail header layout and merchant name handling - #1959

Merged
MusabShakeel576 merged 1 commit into
qafrom
claude/fix-merchant-card-display-xL5yx
Apr 19, 2026
Merged

Improve transaction detail header layout and merchant name handling#1959
MusabShakeel576 merged 1 commit into
qafrom
claude/fix-merchant-card-display-xL5yx

Conversation

@MusabShakeel576

Copy link
Copy Markdown
Contributor

Summary

This PR refactors the transaction detail header layout to use absolute positioning for better centering and improves merchant name handling by trimming whitespace.

Key Changes

  • Header Layout: Changed the back button container from justify-between to justify-center with absolute positioning for the back button, removing the need for a spacer View
  • Merchant Name: Added .trim() calls to merchant name and description to handle whitespace properly
  • Merchant Location: Changed text alignment from left-aligned (ml-10) to center-aligned to match the new header layout

Implementation Details

  • The back button is now positioned absolutely on the left while the title remains centered, creating a cleaner and more flexible layout
  • Merchant name fallback chain now properly handles empty strings by trimming whitespace before checking for falsy values
  • Location text alignment is now consistent with the centered header design

https://claude.ai/code/session_01T6zZc5Zr1FCcFV89aBkEJN

The card purchase activity header used justify-between, which offset
the merchant title based on the ChevronLeft width, and the location
was pinned with ml-10. Absolutely position the back button so the
title truly centers, and drop the spacer. Also center the location
text and trim merchant_name/description so stray whitespace from the
upstream data doesn't misalign the header.
@vercel

vercel Bot commented Apr 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Apr 19, 2026 9:27am
solid-app-staging Ignored Ignored Apr 19, 2026 9:27am

Request Review

@MusabShakeel576
MusabShakeel576 merged commit 9dcdf6e into qa Apr 19, 2026
4 of 5 checks passed
@MusabShakeel576
MusabShakeel576 deleted the claude/fix-merchant-card-display-xL5yx branch April 19, 2026 09:28
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.

2 participants