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

Otters - Amel Ntamark #111

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

amelntamark
Copy link

Personal Portfolio Site

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Did you have to resolve any issues when running the HTML Validator? If so, what were they? I have not tried it yet.
Why is it important to consider and use semantic HTML? It makes the code easier to read and more accessible.
How did you decide to structure your CSS? I used flex elements and lots of googling to get the elements to look the way I wanted.
What was the most challenging piece of this assignment? Moving divs around the page.
Describe one area that you gained more clarity on when completing this assignment That I care much more about what seems to work in CSS as opposed to why it works.
Optional
Did you deploy to GitHub Pages? If so, what is the URL to your website?

Copy link

@kaidamasaki kaidamasaki left a comment

Choose a reason for hiding this comment

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

Great job!

I left one small note but everything else looks good. 😄

Well done!

Comment on lines +28 to +31
favorite color: purple<br><br>
favorite show: the good place<br><br>
favorite song: <iframe style="border-radius:12px" src="https://open.spotify.com/embed/track/29d0nY7TzCoi22XBqDQkiP?utm_source=generator" width="100%" height="80" frameBorder="0" allowfullscreen="" allow="autoplay; clipboard-write; encrypted-media; fullscreen; picture-in-picture"></iframe>

Choose a reason for hiding this comment

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

Style: Using a ul or simple paragraphs instead of having a bunch of brs would be cleaner.

From MDN:

Creating separate paragraphs of text using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element, but not any content contained within <br>s. This can be a confusing and frustrating experience for the person using the screen reader.

To read more: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/br

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for sharing this info Kaida! ✨

<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Amel Ntamark | About</title>
<meta name="description" content="about amel ntamark and how to contact" />

Choose a reason for hiding this comment

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

Nice use of meta tags. 😃

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