-
Notifications
You must be signed in to change notification settings - Fork 0
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
add sponsors #15
add sponsors #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some critical changes are needed, but I think we're on the right direction!
src/components/Sponsors.astro
Outdated
|
||
<div class="sponsors-container"> | ||
<div class="image-container"> | ||
<img src="/src/graphics/sponsors/citris.webp" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To add images, Astro recommends importing them via JavaScript so they can be optimized. Feel free to reference the official docs here or check out some existing code in the Hero component of main branch.
Additionally, please add an alt
attribute to describe the image! This will let users know what the image is supposed to be in case they cannot access them.
src/components/Sponsors.astro
Outdated
h4 { | ||
color: #3d3833; | ||
font-weight: 400; | ||
letter-spacing: 0.05rem; | ||
} | ||
h2 { | ||
color: #3d3935; | ||
font-weight: 500; | ||
b { | ||
color: #3d3935; | ||
font-weight: 600; | ||
} | ||
} | ||
p { | ||
color: #3d3833; | ||
font-weight: 500; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've defined all these in the _global.scss
file! You can just use the tags themselves and the styles should automatically apply!
src/components/Sponsors.astro
Outdated
<section id="sponsors"> | ||
<div class="header-container"> | ||
<h4>SPONSORS & PARTNERS</h4> | ||
<h2>A round of applause for our <b>past sponsors</b></h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
styling: the <h2>
here should be <h3>
instead (which is a different font), and <b>
should be <h2>
* update hero * update text styles, minor adjustment to hero layout * remove changes to env.d.ts * tell prettier to ignore env.d.ts * tell lint to ignore env.d.ts * update scroll behavior and padding * microscopic edits * add bouncing arrow
* create skeleton * change div to section * fill out Our Mission + Vision * add and use b * clean up / remove comments * check style (prettier) * remove b from global styles * create div for mission and vision
What's new?
Dev
Dez
Linear Task
https://linear.app/hack-for-impact-2024/issue/HFI-17/sponsors-past-sponsors-section