Skip to content

Sammajayi/connect a wallet fixes - #135

Open
sammajayi wants to merge 1 commit into
GoodDollar:plan/connect-a-wallet-widgetfrom
sammajayi:sammajayi/connect-a-wallet-fixes
Open

Sammajayi/connect a wallet fixes#135
sammajayi wants to merge 1 commit into
GoodDollar:plan/connect-a-wallet-widgetfrom
sammajayi:sammajayi/connect-a-wallet-fixes

Conversation

@sammajayi

Copy link
Copy Markdown

Description

Redesigned the connect-wallet widget UI following the design reference in #113 in reference to the main pr in #116, executed following the plan in #114 . Fixed crash on error/unsupported states, addressed overflow on small viewports, and updated component styling to match the design system.

PR about #116

How Has This Been Tested?

  • Ran pnpm test: demo tests/widgets/connect-a-wallet-widget - all 8 Playwright state tests pass
  • Manual testing in Storybook for visual verification
  • Verified build and lint pass

Checklist:

  • PR title matches follow: (Feature|Bug|Chore) Task Name
  • My code follows the style guidelines of this project
  • I have followed all the instructions described in the initial task (check Definitions of Done)
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added reference to a related issue in the repository
  • I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
  • I have added screenshots related to my pull request (for frontend tasks)
  • I have pasted a gif showing the feature.
  • @mentions of the person or team responsible for reviewing proposed changes

- Fix missing Alert import crash on error/unsupported states
- Remove unused address prop from ChainLinkRow (lint fix)
- Restructure PrimaryIdentityCard layout (shield/text/copy)
- Reduce GlowCard shadow opacity and radius
- Update unlink SVG path to match lucide icon
- Add flexWrap to ChainLinkRow for 320px viewport support
- Reduce input/button sizes for small viewports
- Center address in verified identity box with wide gap
- Adjust heading spacing and checkmark position
- Update Playwright test screenshots
@sammajayi
sammajayi force-pushed the sammajayi/connect-a-wallet-fixes branch from bbfde2f to ed45fce Compare July 30, 2026 11:24
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