Skip to content

feat: responsive NFT grid layout and apiClient test coverage - #529

Merged
Akatenvictor merged 1 commit into
AudioBitsStellar:mainfrom
WISDOM-WOKE22:feat/responsive-grid-modal-toast-api-tests
Sep 28, 2026
Merged

Akatenvictor merged 1 commit into
AudioBitsStellar:mainfrom
WISDOM-WOKE22:feat/responsive-grid-modal-toast-api-tests

Conversation

@WISDOM-WOKE22

Copy link
Copy Markdown
Contributor

Summary

  • Replaced fixed grid-cols-* with CSS Grid auto-fill/minmax(280px, 1fr) for responsive NFT collection layout (1 col mobile → 2 cols tablet → 3-4 cols desktop)
  • Enforced 280px minimum card width and prevented horizontal overflow at all breakpoints
  • Added apiClient tests for 4xx no-retry behavior, 502 retry with backoff, and 503 retry with backoff

Test plan

  • Verify NFT grid displays 1 column below 640px
  • Verify 2 columns at 640-1024px
  • Verify 3-4 columns above 1024px
  • Confirm no horizontal overflow at 375px, 768px, 1024px, 1440px
  • Run npx vitest run tests/lib/apiClient.test.ts — all 11 tests pass

Closes #80, Closes #81, Closes #89, Closes #147

- Replace fixed grid columns with CSS Grid auto-fill/minmax(280px, 1fr)
  for responsive 1→2→3-4 column layout across breakpoints
- Enforce 280px minimum card width and prevent horizontal overflow
- Add tests for 4xx no-retry behavior, 502 and 503 retry with backoff

Closes AudioBitsStellar#80, Closes AudioBitsStellar#81, Closes AudioBitsStellar#89, Closes AudioBitsStellar#147
@vercel

vercel Bot commented Sep 28, 2026

Copy link
Copy Markdown

@WISDOM-WOKE22 is attempting to deploy a commit to the akatenvictor's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Sep 28, 2026

Copy link
Copy Markdown

@WISDOM-WOKE22 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Akatenvictor
Akatenvictor merged commit 530fd6e into AudioBitsStellar:main Sep 28, 2026
2 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants