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

Complete react week 1 tasks and added screenshots #6

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

noushinjalali
Copy link

This pull request contains the completion of the tasks assigned for Week 1 of the React module. The tasks involved setting up and enhancing the basic structure of the Galactica space travel website using React and Next.js.

Tasks Completed:

Footer Component:

Imported and used the Footer component in the RootLayout to ensure consistent footer display across the app.
Added social media links to the footer, including Facebook, Instagram, Tiktok, and LinkedIn.
About Us Page:

Created the "Our Values" and "Our Crew" sections.
Used the provided descriptions for each value and crew member.
Displayed images of crew members with appropriate alt text for accessibility.
Screenshots:

Included screenshots of the website displaying the completed "About Us" page and footer section as required.

Anousheh-Ansari Douglas Hurley Mark Shuttleworth Victor Glover

Copy link
Collaborator

@ddobby94 ddobby94 left a comment

Choose a reason for hiding this comment

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

Hey hey,

You can always let us know if you are struggling with the HW.
I'm not sure if you didn't understand enough of the React or it was a time pressure. Let us know if we can help you somehow.

@@ -22,7 +24,17 @@ const OurCrew = () => {
// Use the pictures from /public/crew
// Some inspiration ideas found in /data/inspiration_about_us
return (
<p> ADD OUR CREW HERE </p>
<div>
<p>Meet the best space travelers and scientist in the world.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

It is very important to use proper indentation (spaces/tabs) to show what is the parent and what are the children.

  <mainParent>
    <level2>
      <level3 />
    <level2>
    <level2Sibling>
    </level2Sibling>
  </mainParent>

<p> ADD OUR CREW HERE </p>
<div>
<p>Meet the best space travelers and scientist in the world.</p>
<img src="/crew/image-anousheh-ansari.png" alt="Anousheh Ansari" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

You are missing the width and height properties from the img (either as a html prop here, or from css), that is why you have those huge images 😄

<p> ADD OUR Partners HERE </p>
<div>
<h2>Our Partners</h2>
<p>Collaboration with the best space agencies and tech companies.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

It seems like you missed out on adding the logos for the partners

Content

Use the content saved in the /public/business_partners folder!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also this section was not added to the page

<div>
<h2>Our Partners</h2>
<p>Collaboration with the best space agencies and tech companies.</p>
</div>
);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

These components should be moved from a different file.

<p>Anousheh Ansari</p>
<img src="/crew/image-douglas-hurley.png" alt="Douglas Hurley" />
<p>Douglas Hurley</p>
<img src="/crew/image-victor-glover.png" alt="Victor Glover" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

This alt="..." text is not providing any extra info for the non-sighted users, so you can either change it or remove it.

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