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

add sponsors #15

Conversation

christophertorres1
Copy link
Contributor

What's new?

  • added sponsors section

Dev
Screenshot 2024-01-10 at 11 22 38 PM

Dez
Screenshot 2024-01-10 at 11 31 33 PM

Linear Task

https://linear.app/hack-for-impact-2024/issue/HFI-17/sponsors-past-sponsors-section

Copy link
Collaborator

@jinkang-0 jinkang-0 left a 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!


<div class="sponsors-container">
<div class="image-container">
<img src="/src/graphics/sponsors/citris.webp" />
Copy link
Collaborator

@jinkang-0 jinkang-0 Jan 11, 2024

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.

Comment on lines 46 to 63
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;
}
}
Copy link
Collaborator

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!

<section id="sponsors">
<div class="header-container">
<h4>SPONSORS & PARTNERS</h4>
<h2>A round of applause for our <b>past sponsors</b></h2>
Copy link
Collaborator

@jinkang-0 jinkang-0 Jan 11, 2024

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>

jinkang-0 and others added 7 commits January 11, 2024 15:35
* 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
@christophertorres1 christophertorres1 deleted the christophertorres/hfi-17-sponsors-past-sponsors-section branch January 11, 2024 23:58
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