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

[feat] create about section (Our Vision, The Mission) #13

Merged
merged 8 commits into from
Jan 11, 2024

Conversation

ccatherinetan
Copy link
Contributor

@ccatherinetan ccatherinetan commented Jan 11, 2024

What's new?

  • creates the 2nd half of about i.e. Our Vision, The Mission
  • adds b in _global.css

N.B.

  • The description next to Our Vision and The Mission have a font-weight of 600 in the figma description, but I kept it as a normal p tag (font-weight = 500), since it's visually more similar to the design.
  • the captions next to the images under The Mission: current implementation has them flush against the outer div, but the figma version seems to have slightly more padding on the right.

Current:
Screen Shot 2024-01-10 at 11 08 12 PM
Compare to Figma:
Screen Shot 2024-01-10 at 11 09 49 PM

Linear Task

https://linear.app/hack-for-impact-2024/issue/HFI-14/about-section

@ccatherinetan ccatherinetan changed the title [feat] create skeleton [feat] create about section Jan 11, 2024
@ccatherinetan ccatherinetan changed the title [feat] create about section [feat] create about section (Our Mission, Our Values) Jan 11, 2024
@ccatherinetan ccatherinetan changed the title [feat] create about section (Our Mission, Our Values) [feat] create about section (Our Vision, The Mission) Jan 11, 2024
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.

looks great so far! added some nits for review.

src/styles/_global.scss Outdated Show resolved Hide resolved
src/components/About.astro Outdated Show resolved Hide resolved
src/components/About.astro Outdated Show resolved Hide resolved
src/components/About.astro Outdated Show resolved Hide resolved
src/components/About.astro Show resolved Hide resolved
@ccatherinetan ccatherinetan force-pushed the catherine/hfi-14-about-section branch from cd00e4f to 4357938 Compare January 11, 2024 18:50
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.

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.

Comment on lines 45 to 53
section {
display: flex;
flex-direction: column;
align-items: center;
text-align: center;
max-width: 58.625rem;
margin: 0 auto;
padding-top: 5rem;
}
Copy link
Collaborator

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.

Copy link
Contributor Author

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

@ccatherinetan ccatherinetan merged commit 3a05605 into main Jan 11, 2024
2 checks passed
me-liu pushed a commit that referenced this pull request Jan 11, 2024
* 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 pushed a commit that referenced this pull request Jan 11, 2024
* 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
jinkang-0 added a commit that referenced this pull request Jan 12, 2024
* 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]>
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.

2 participants