Skip to content

vtool button and event summary support added on events page - #12

Merged
Anurajx merged 2 commits into
IEEE-MSIT:mainfrom
Anurajx:main
Aug 24, 2026
Merged

vtool button and event summary support added on events page#12
Anurajx merged 2 commits into
IEEE-MSIT:mainfrom
Anurajx:main

Conversation

@Anurajx

@Anurajx Anurajx commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Data for vtools is yet to be added


Summary by cubic

Adds an event details modal on the Events page and vTools link support to improve event discovery and external references. Previously, the card’s primary button navigated to register/join; now on the Events page the card and primary button open a modal, with links available inside. Other pages keep the old behavior.

  • EventCard: accepts optional onOpenDetails; when provided, the card becomes clickable and the primary button shows “View Details.” Click handlers stop propagation to avoid accidental modal toggles. No change where onOpenDetails is not used.
  • New EventDetailsModal: shows description, dates, venue, category, organizers, prizes, contacts, and action links (register/livestream/URLs). Closes on Escape/backdrop and uses a portal to lock scroll.
  • vTools support: adds optional vtool field to Event and a getVtoolHref utility to normalize URLs; modal shows an “Open in vTools” button when present. Action: populate vtool URLs in event data when available.
  • Minor: hero typography updated for emphasis (bold instead of italic).

Written for commit 158406c. Summary will update on new commits.

Review in cubic

@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

@Anurajx is attempting to deploy a commit to the nithin0620's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Anurajx
Anurajx merged commit c0ff924 into IEEE-MSIT:main Aug 24, 2026
1 of 2 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

Development

Successfully merging this pull request may close these issues.

1 participant