Skip to content

Conversation

@ManasAgrawal05
Copy link
Collaborator

@ManasAgrawal05 ManasAgrawal05 commented Jan 9, 2025

Sponsors page no longer shows our old corporate package. Instead it has our 3 sponsors on it on square cards. This same row is present at the bottom of our homepage.
I added all of 2024 officers as well. I made the card information more flexible in case an officer doesn't have linked in or an email. Email refs are still broken, and I'm not sure why. They were broken on the original website and I thought the links were just wrong but they seem to be correct.
image

image image

@github-actions
Copy link

github-actions bot commented Jan 9, 2025

Visit the preview URL for this PR (updated for commit 0b266e5):

https://eclairrobotics--pr24-manas-sponsors-updat-46j8mrsv.web.app

(expires Sun, 19 Jan 2025 20:55:16 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 150807af4c601ea757f7d0a4dd845c67c6fe4369

Copy link
Contributor

@vishalkantharaju vishalkantharaju left a comment

Choose a reason for hiding this comment

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

Overall looks solid, thank you for the good work!! Just had a couple of small comments

.currentSponsorCard:hover {
transform: scale(1.05);
box-shadow: 0 2em 2em rgba(0,0,0, 0.2);
cursor: pointer;
Copy link
Contributor

Choose a reason for hiding this comment

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

Since the cards aren't currently clickable, thoughts on taking this out for now? Might lead to confusion with users


<div style = {{height:"5rem"}}></div>
{/* <div style={{display:'flex', justifyContent:'center', alignContent:'center'}}> */}
<div style={{display:'flex', justifyContent:'center', alignContent:'center', width:'100%'}}>
Copy link
Contributor

Choose a reason for hiding this comment

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

For the box stating "We are always trying to provide our members with the most cutting-edge robotics hardware available and would greatly appreciate any potential sponsorships. If you are interested in sponsoring UT ECLAIR, please contact us at the email below and we can send you our sponsor packet!", thoughts on making it a bit less clear? The dots in the background seem like they could make this harder to read

email='sahanagana@utexas.edu' topAlignImage={false}/>
name='Cameron Cox'
position='Corporate Director'
img={cameronImg} bio="Hey y'all, I'm Cameron. I'm a computer Science Major with a passion for robotics and in my freetime I play D&D, piano and cook a lot."
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be possible to adjust the spacing a bit? It seems that cards with less text end up having a bit more empty space (pic attached).
Screenshot 2025-01-10 at 11 10 27 PM


return (
<div className='currentSponsorRow'>
<CurrentSponsorCard img={serviceNow} color='rgb(30, 59, 61)'/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is on me but can we make a colors resource file in one of the future pushes

@ManasAgrawal05 ManasAgrawal05 merged commit e7bd272 into main Jan 13, 2025
2 checks passed
@ManasAgrawal05 ManasAgrawal05 deleted the manas/sponsors-update branch January 13, 2025 23:41
@ManasAgrawal05 ManasAgrawal05 restored the manas/sponsors-update branch January 13, 2025 23:57
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.

3 participants