Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

NW5-Manchester-Sandra Duarte-HTML/CSS -week-3 - #395

Open
Sandra-Duarte wants to merge 2 commits into
CodeYourFuture:masterfrom
Sandra-Duarte:master
Open

NW5-Manchester-Sandra Duarte-HTML/CSS -week-3#395
Sandra-Duarte wants to merge 2 commits into
CodeYourFuture:masterfrom
Sandra-Duarte:master

Conversation

@Sandra-Duarte

@Sandra-Duarte Sandra-Duarte commented Aug 12, 2022

Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name: Sandra Duarte
  • Your City: Manchester
  • Your Slack Name:Sandra Duarte

Homework Details

  • Module: HTML/CSS
  • Week:3

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@Ara225 Ara225 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.

I really love the look and feel of this, you've obviously taken time with the colors and font, they look really good. The responsive side of things isn't really working that well sadly, but it's good that you made a start, I can see you understand the basics of media queries

Comment thread index.html
<!--portfolio-->
<section class="portfolio">

<img class="images1" src="/img/pexels-daria-shevtsova.jpg" alt="#">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It would be best to add alt text explaining what the the image is if possible :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I really love the look and feel of this, you've obviously taken time with the colors and font, they look really good. The responsive side of things isn't really working that well sadly, but it's good that you made a start, I can see you understand the basics of media queries

Thank a lot for your feedback.

Comment thread style.css Outdated
cursor: pointer;
}
.card-social{
border: 1px; solid #b1b4b7;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Small mistake

Comment thread style.css



/*grid style

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Best not to leave commented out code in if possible

Comment thread style.css

/*media queries*/

@media ( max-width: 540px) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I actually quite like the way the buttons go on smaller screens, it's not what the design asked for but it's still really nice in it's own way, could have done with a bit more work to add a middle breakpoint and make the smallest breakpoint look like the mobile veiw

Comment thread style.css
text-align: center;
}

.main-paragrafo p{

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good descriptive names :)

Comment thread index.html
</ul>
</nav>
<!--end header-->
<!-- top container-->

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could have done with a opening main tag here, but your sematic HTML is good otherwise :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants