Skip to content

Improve Event page Performance#7621

Open
AnkitRewar11 wants to merge 2 commits intolayer5io:masterfrom
AnkitRewar11:fix/event-page-performance
Open

Improve Event page Performance#7621
AnkitRewar11 wants to merge 2 commits intolayer5io:masterfrom
AnkitRewar11:fix/event-page-performance

Conversation

@AnkitRewar11
Copy link
Copy Markdown
Contributor

Description

This PR improves the Events page performance and accessibility based on PageSpeed Insights recommendations from issue #6929.

Fixes #6929

What was wrong?

  • Card links had no readable name for screen readers
  • rel attribute had a typo (oopener instead of noopener)
  • Event images were missing width and height causing layout shifts
  • Page was missing a <main> landmark element
  • Filter buttons had no accessibility attributes

What did I fix?

  • ✅ Added aria-label to all card links so screen readers can identify them
  • ✅ Fixed rel="noreferrer noopener" typo
  • ✅ Added explicit width and height to event images to reduce CLS (layout shift)
  • ✅ Wrapped page content in <main> landmark for better navigation
  • ✅ Added role="group" and aria-pressed to filter buttons for keyboard accessibility

Lighthouse Score Improvements (Mobile)

Metric Before After Change
Performance 91 93 +2 ✅
Accessibility 86 91 +5 ✅
Best Practices 100 100 — ✅
SEO 92 92 — ✅

Screenshot

Screenshot

Signed-off-by: ankitrewar11 <ankitrewar11@gmail.com>
@AnkitRewar11 AnkitRewar11 force-pushed the fix/event-page-performance branch from 66217cf to 7cf252e Compare April 11, 2026 22:54
Signed-off-by: Ankit Rewar <171806101+AnkitRewar11@users.noreply.github.com>
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.

[ Performance ] Improve Event page Performance

1 participant