Skip to content

Feature/941 958 lazy image focus trap#995

Merged
Yunusabdul38 merged 3 commits into
Agora-Events:mainfrom
tali-creator:feature/941-958-lazy-image-focus-trap
Jun 30, 2026
Merged

Feature/941 958 lazy image focus trap#995
Yunusabdul38 merged 3 commits into
Agora-Events:mainfrom
tali-creator:feature/941-958-lazy-image-focus-trap

Conversation

@tali-creator

Copy link
Copy Markdown
Contributor

close #941
closes #939

Agora-Events#939 Agora-Events#941 Agora-Events#958)

- LazyImage: blur-up placeholder, IntersectionObserver swap, CSS transition
- EventCard: replaced img with LazyImage
- useFocusTrap: custom hook trapping Tab/Shift+Tab, restores trigger focus on close
- TicketModal: wired useFocusTrap, role=dialog, aria-modal, aria-labelledby/describedby
- Tests: focus-trap-modal (4) and lazy-image (4) all passing
- Fix: moved supportsIO check inside component to avoid module-level evaluation
- Fix: duplicate jsdom devDependency removed from package.json
- Chore: cargo fmt across root, server, contract workspaces
- Chore: clippy fixes in src/location.rs (manual_range_contains, dead_code)
- Chore: suppress unexpected_cfgs from soroban macro via Cargo.toml lints
@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

@tali-creator is attempting to deploy a commit to the oseh-svg's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 29, 2026

Copy link
Copy Markdown

@tali-creator 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

@Yunusabdul38 Yunusabdul38 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Thanks for your contribution

@Yunusabdul38 Yunusabdul38 merged commit 77b9440 into Agora-Events:main Jun 30, 2026
1 check failed
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.

## #958 – FRONTEND: Add focus‑trap to all modal dialogs FRONTEND: Add LazyImage component with IntersectionObserver

2 participants