Skip to content

london-class-7/ramla-jama - #143

Closed
ramla199 wants to merge 1 commit into
CodeYourFuture:masterfrom
ramla199:master
Closed

london-class-7/ramla-jama#143
ramla199 wants to merge 1 commit into
CodeYourFuture:masterfrom
ramla199:master

Conversation

@ramla199

Copy link
Copy Markdown

Your Details

  • Your Name:Ramla Jama
  • Your City:London
  • Your Slack Name:Ramla

Homework Details

  • Module:HTML/CSS
  • Week:Week 2

@Behnazz Behnazz added the London class 7 start December 2020 label Dec 13, 2020

@Behnazz Behnazz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a very good website. Welldone :) .
Your media query needs a bit more work.
Just some minor comments to improve it.

Comment thread css/style.css
Comment on lines +178 to +179
order: 1;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
order: 1;
}
order: 1;
margin-right: 120px;
}

Comment thread css/style.css

nav {
display: flex;
justify-content: space-around;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
justify-content: space-around;
justify-content: space-between;

Comment thread css/style.css

.nav-wrapper {
display: flex;
flex-direction :row;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

by default flex direction is flex row

Suggested change
flex-direction :row;

Comment thread css/style.css
Comment on lines +43 to +46
flex-wrap : nowrap;
justify-content:flex-end;
align-items:flex-start;
align-content:space-between;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
flex-wrap : nowrap;
justify-content:flex-end;
align-items:flex-start;
align-content:space-between;
flex-wrap : nowrap;
justify-content:flex-end;
align-items:flex-start;
align-content:space-between;

Do you need these? What will happen if we don't have them?

Comment thread css/style.css
padding: 25px;
background-size: cover;
background-repeat: no-repeat;
background-size: 90% 100%;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
background-size: 90% 100%;

Comment thread index.html
Comment on lines +29 to +30


Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change

Comment thread index.html


</header>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change

Comment thread index.html
</header>

<main>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change

Comment thread index.html
Comment on lines +40 to +44

<br>
<br>
<br>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
<br>
<br>
<br>

It is not a good practice to have
to achieve a styling goal. It is better to use CSS to do the styling

Comment thread index.html
</section>
<p class= "copy-write">&copy Karma Mobility, Inc.</p>
</footer>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change

Comment thread css/style.css
Comment on lines +183 to +184
}
} No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
}
}
}
.nav-wrapper {
display: none;
}
nav {
padding: 0;
}
#info {
height: 60vh;
}
}

@Behnazz Behnazz added the reviewed A mentor has reviewed this PR label Dec 17, 2020
@github-actions

Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in six weeks. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label May 14, 2021
@github-actions github-actions Bot closed this May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

London class 7 start December 2020 reviewed A mentor has reviewed this PR Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants