Skip to content

northwest4/declan-heritage - #199

Closed
Declanhs wants to merge 5 commits into
CodeYourFuture:masterfrom
Declanhs:master
Closed

northwest4/declan-heritage#199
Declanhs wants to merge 5 commits into
CodeYourFuture:masterfrom
Declanhs:master

Conversation

@Declanhs

@Declanhs Declanhs commented Jun 6, 2021

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:declan
  • Your City:northwest
  • Your Slack Name:declanhs

Homework Details

  • Module:html/css
  • Week:2

Notes

  • What did you find easy?

  • not much

  • What did you find hard?

  • most of it

  • What do you still not understand?

  • flex

  • Any other notes?

Declanhs added 5 commits May 25, 2021 05:54
background wont show :S
Just footer left to do and cleaning up responsiveness. (also need to covert svg files that should be fun!)
Done

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

Hey @Declanhs !
This is absolutely amazing / beautiful. Almost identical as the photo 😍.
Just some miniscule minor feedback below.

Comment thread css/style.css
Comment on lines +58 to +60
button {
background-color: rgb(240, 128, 0);
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Might want to add a padding property and a border-radius property to match the photo.

Comment thread css/style.css
Comment on lines +28 to +30
.nav-item:hover {
color: orange;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Might want to add cursor: pointer; here to have a hand icon to match photo

@github-actions

github-actions Bot commented Aug 3, 2021

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 Aug 3, 2021
@github-actions github-actions Bot closed this Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants