Skip to content

Comments

Miguel Created mission component#32

Open
migueljbeltran wants to merge 3 commits intomainfrom
miguel-mission
Open

Miguel Created mission component#32
migueljbeltran wants to merge 3 commits intomainfrom
miguel-mission

Conversation

@migueljbeltran
Copy link
Collaborator

@migueljbeltran migueljbeltran commented Feb 17, 2026

added to homepage for testing. got styling from figma css, put everything in section container.

Description

created mission component, placed it in home page for testing. fills the entire page

Closes #23

image

Checklist:

after creating your PR please mark all of these:

  • Global colors were used (let us know if any are missing)
  • Text/images/paths are not hard coded (passed in by props)

added to homepage for testing. got styling from figma css, put everything in section container.
@migueljbeltran migueljbeltran self-assigned this Feb 17, 2026
@migueljbeltran migueljbeltran linked an issue Feb 17, 2026 that may be closed by this pull request
@migueljbeltran migueljbeltran changed the title Created mission component Miguel Created mission component Feb 17, 2026
padding: 0.625rem;
flex-direction: column;
justify-content: center;
align-items: flex-start;
Copy link
Collaborator

Choose a reason for hiding this comment

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

it's better if align-items is center, it'll fit better in general on various screen sizes


.header {
display: flex;
height: 16.3125rem;
Copy link
Collaborator

Choose a reason for hiding this comment

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

same thing here, instead of using height this could definitely be like padding-bottom or maybe a margin

.container {
display: flex;
width: 100vw;
height: 30.1875rem;
Copy link
Collaborator

Choose a reason for hiding this comment

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

use padding over width height

.content {
display: flex;
width: 59.75rem;
height: 16.3125rem;
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think this is trying to set the right side? i'm a little confused, whatever is setting the right side text sizing should probably be moved to .textBlock and also set a max-width for it

Copy link
Collaborator

Choose a reason for hiding this comment

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

can you also move your component to localhost:3000/test you will probably have to create the test page

}

.title {
color: var(--white);
Copy link
Collaborator

Choose a reason for hiding this comment

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

can you make all white texts var(--off-white)

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.

Mission

2 participants