-
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
[feat] create about section (Our Vision, The Mission) #13
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.
looks great so far! added some nits for review.
cd00e4f
to
4357938
Compare
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.
Everything looks great Catherine! I added a suggestion for the About section overall, but you can talk with Melissa to decide who should implement that. The Vision and Mission section should be good to go.
src/components/About.astro
Outdated
section { | ||
display: flex; | ||
flex-direction: column; | ||
align-items: center; | ||
text-align: center; | ||
max-width: 58.625rem; | ||
margin: 0 auto; | ||
padding-top: 5rem; | ||
} |
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.
For the purpose of the image carousel, it's best to make this width: 100%
and contain the Vision and Mission sections inside a smaller child div
.
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.
MMM this makes sense -- lemme do that
* 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
* 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
* feat(faq): adds faq section * [feat] add button component (#8) * add graphics, button * update global button style * update button styles * [feat] add nav bar (#9) * update navbar, h5 global style * update navbar height * [feat] add footer (#12) * add footer * remove edit to env.d.ts * use astro image for optimization * remove edit to env.d.ts * [feat] add hero section (#11) * 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 * add padding-top to Contact section (#14) * fix(types): moves question type definition to FAQDropdown component * [feat] create about section (Our Vision, The Mission) (#13) * 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 * feat(faq): adds faq section * fix(types): moves question type definition to FAQDropdown component * refactor: addresses review comments * refactor(faq): adjusts container width, updates keydown functionality --------- Co-authored-by: Jinkang Fang <[email protected]> Co-authored-by: ccatherinetan <[email protected]>
What's new?
adds b in _global.cssN.B.
Current:
Compare to Figma:
Linear Task
https://linear.app/hack-for-impact-2024/issue/HFI-14/about-section