Skip to content

Design event - #5899

Open
felixvik21 wants to merge 6 commits into
interest-group-profrom
design-event
Open

Design event#5899
felixvik21 wants to merge 6 commits into
interest-group-profrom
design-event

Conversation

@felixvik21

Copy link
Copy Markdown
Contributor

Make design for interest group event page

Result

If you've made visual changes, please check the boxes below and include images showing the changes. Descriptions are appreciated.

  • Changes look good on both light and dark theme.
  • Changes look good with different viewports (mobile, tablet, etc.).
  • Changes look good with slower Internet connections.

Caution

Make sure your images do not contain any real user information.

image

Testing

  • I have thoroughly tested my changes.

Please describe what and how the changes have been tested, and provide instructions to reproduce if necessary.


Resolves ... #5898

Copilot AI review requested due to automatic review settings March 3, 2026 20:00
@github-actions github-actions Bot added the review-needed Pull requests that need review label Mar 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR implements the visual design for the interest group event detail page (/interest-events/:interestEventId), addressing issue #5898. It creates a new page layout with multiple sub-sections (attendees, attend/registration, organizer, and admin) displayed in a two-column split view.

Changes:

  • Adds a new +Page.tsx event detail page with hardcoded placeholder layout (header image, event info, description, and sidebar sections)
  • Introduces four sub-section components (AttendeesSection, AttendSection, OrganizerSection, AdminSection) each with their own CSS module
  • Adds shared box styles and a new --font-size-xxl CSS variable to the global design system

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 18 comments.

Show a summary per file
File Description
+Page.tsx Main page layout with hardcoded placeholder content for the interest event detail view
AdminSection.tsx Admin panel stub with a chevron arrow icon
AttendSection.tsx Registration/sign-up box with attend button and deadline info
AttendeesSection.tsx Attendee list section with avatar stack and progress bar
OrganizerSection.tsx Organizer and contact info box with hardcoded names
adminSection.module.css Minimal CSS for arrow icon sizing
attendSection.module.css Styles for dot indicator, attend button, and deadline info
attendeesSection.module.css Styles for stacked avatar icons and progress bar
boxes.module.css Shared styles for the right-side info box cards
interestEventDetail.module.css Main page layout styles with Google Fonts import
organizerSection.module.css Text-align style for organizer list
variables.css Adds --font-size-xxl: 2.5rem design token

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lego-webapp/pages/interest-events/@interestEventId/AttendSection.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/attendSection.module.css Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/+Page.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/OrganizerSection.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/AttendSection.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/AdminSection.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/AttendSection.tsx Outdated
Comment thread lego-webapp/pages/interest-events/@interestEventId/interestEventDetail.module.css Outdated

Copilot AI commented Mar 10, 2026

Copy link
Copy Markdown

@felixvik21 I've opened a new pull request, #5905, to work on those changes. Once the pull request is ready, I'll request review from you.

@celiacisternas celiacisternas 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.

Looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-needed Pull requests that need review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants