Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Events page #265

Open
wants to merge 26 commits into
base: main
Choose a base branch
from
Open

Events page #265

wants to merge 26 commits into from

Conversation

fatmahussein
Copy link
Collaborator

@fatmahussein fatmahussein commented Jan 31, 2025

Changes Implemented in this section:

πŸ“… An interactive calendar to display upcoming events.
- βœ… Integrated FullCalendar for displaying upcoming events dynamically.
-
🎟 A past meetups section with summaries and links to resources.
- βœ… Designed a past meetups section styled as per the Figma mockup.

πŸ“± Responsive design ensuring functionality across desktop and mobile.

βœ… Ensured hover/click interactions for detailed event information.
βœ… Optimized performance with lazy loading for past meetups.

Notes
Further enhancements (if needed) can be added based on feedback.
Open to suggestions on improving performance or interactivity.
Screenshot (172)
173

@fatmahussein fatmahussein linked an issue Jan 31, 2025 that may be closed by this pull request
package.json Outdated Show resolved Hide resolved

.events-cards::-webkit-scrollbar-thumb {
background-color: #e8e8cd;
height: 10rem !important;
Copy link
Collaborator

Choose a reason for hiding this comment

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

I love the way you're handling it 🩷. Later on we'll see how to remove the !important from the project. Its use is considered bad practice.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Agree with Mayra, I use fullcalendar in fflow, and I regularly have to spend time tracking down the magic incantation that means I can avoid important. Happy for this PR to merge as is, and fix up the !importants in a separate PR.

@@ -1,48 +1,54 @@
import React from 'react';
import PropTypes from 'prop-types';
import meetup from '../../../assets/images/meetup.jpg';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like this MeetUp page hasn't yet been (fully) styled to match the figma designs, I assume intentional. Happy to keep it for a separate issue.

<div className="page-title-calendar">
<h1 className="font-besley text-2xl font-bold mb-2">Upcoming Events</h1>
<p>
<a href="/join-us">Register </a>to stay updated with upcoming events
Copy link
Collaborator

Choose a reason for hiding this comment

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

suggest add "and be sent meeting invitations"

{loading ? (
<p>Loading events...</p>
) : meetups.length === 0 ? (
<p>No events available</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

❀️

<div
id={event.id}
key={event.id}
className={`event-card ${index % 2 === 0 ? 'yellow' : 'green'}`}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Option to do the striping in CSS with nth-child. It's fine this way too.

}
}

@media (min-width: 850px) and (max-width: 1024px) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are potential styling gaps here betwen 767 and 850px, and betwee 1024 and 1094px. An easy way to resolve is just only one of min or max width, not both.

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.

Events Page
4 participants