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

added current events section to home page #493

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kaylahama
Copy link
Contributor

Fixes #480

Changes

Created a current events section on the home page to summarize events. Played around with a gradient and hover effect for the title to add some color, but the shine hover effect could probably be improved.

Type of change

  • New feature (non-breaking change which adds functionality)

@kaylahama kaylahama requested a review from jamesmwu November 20, 2024 06:08
Copy link
Contributor

@jamesmwu jamesmwu left a comment

Choose a reason for hiding this comment

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

The section looks great! Just have some minor changes for you to fix, let me know if you run into trouble with anything 👍

Also, be sure to resolve any conflicts with main-- there have been some updates on that branch :)

# Get the newest changes from main
git checkout main
git pull

# Apply the changes to your local branch
git checkout kayla/currEventsSection
git merge main

# At this point, you'll probably have to resolve the conflicts in the merge conflict editor. You should see some stuff indicating whether you need to accept the incoming or accept the current.

# After making your changes, repush your branch
git add .
git commit
git push

import '../../styles/Events.css';
import '../../styles/Home.css';
import hfamimg from '../../images/hack-fam-insta.png';
import schoolimg from '../../images/hack-school-insta.jpg';
Copy link
Contributor

Choose a reason for hiding this comment

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

Rename hfamimg and schoolimg to something along the lines of "img1" and "img2" respectively-- this is just so there's more reusability in the future.

padding: 1rem;
border-radius: 8px;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add a CSS selector here for .event-img (which you've already defined in CurrEvents.jsx). We should make sure that the two images are the same size.

.event-title:hover {
background-position: 100% center;
transition: background-position 2s ease;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

At the bottom of this file, we should have a section for media queries to make sure that the images are centered when the screen enters a smaller window size. I've recorded a Loom here to explain better how you can test this locally, if you need some more context:

https://www.loom.com/share/5537c3fd45b74c4095783ac37fd06f52

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.

Add current quarter events to home page
2 participants