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

North West 4- Anza Azam -HTML/CSS Week 1#5

Open
Anza-Azam wants to merge 8 commits into
CodeYourFuture:mainfrom
Anza-Azam:main
Open

North West 4- Anza Azam -HTML/CSS Week 1#5
Anza-Azam wants to merge 8 commits into
CodeYourFuture:mainfrom
Anza-Azam:main

Conversation

@Anza-Azam
Copy link
Copy Markdown

Zoo CSS Challenges attempt.
Modified website with changes.

@Anza-Azam
Copy link
Copy Markdown
Author

Fished off with flex usage

Comment thread zoo-css-challenge/index.html Outdated
</div>
<div class="card">
<h4>Youth and young adults</h4>
<a href="" target="_blank">Find out More</a>
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 - "target="_blank" is correct. You could also add rel="noopener", although that's not really necessary on trusted links. But it's useful to know (you can read more about it here https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types/noopener)

Copy link
Copy Markdown

@KarenPudner KarenPudner left a comment

Choose a reason for hiding this comment

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

Great work! You have covered everything that was required and you resulting webpage looks good.

Comment thread zoo-css-challenge/style.css Outdated
}
button:focus{

background-color:blue;
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 use of both hover and focus.

Comment thread zoo-css-challenge/style.css Outdated

.introduction {
background-color: white;
line-height: 2.5;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The instructions say that most of the text on the page is too close together, so maybe add line-height:2.5 to * or body

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.

thank you Karen, I've now added line-height:2.5 to *

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