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

Aline's first project draft for review #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alineschlu
Copy link

Hey! Here's my very first very simple code for a webpage, looking forward to your feedback :D

Copy link

@esteficodes esteficodes left a comment

Choose a reason for hiding this comment

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

🌈You are off to a great start! Remember to mind the indentation and start trying media queries so that you can see your site beautifully displayed in several devices. Starting from mobile display will be key. 😀

<!DOCTYPE html>
<html>

<head>

Choose a reason for hiding this comment

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

Hi! Nice to meet you! I'm Estefanía and I'll be taking a look at your site's code to see how can we make it functional and shiny.👋😀.

<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Aline's page</title>
<link rel="stylesheet" type="text/css" href="style.css">

Choose a reason for hiding this comment

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

Maybe you forgot to close the head tag?

Comment on lines +10 to +16
<!--
Write your code here inside the body to make it appear in the browser.
Below is an example on how to use an image tag. Copy it and add it in the body, but outside the comment tag if you want to see how it looks like on the website. Then you can change the size and other styling for it in the css file.

<img src="example-recipe-background.png" alt="Banana Bread">

-->

Choose a reason for hiding this comment

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

Remember to erase all this comments once you are done.😀

Comment on lines +17 to +28
<h1 class="top-heading">Aline Schlunegger</h1>
<h2 class="second-heading">Film-Online-Marketing-Redaktion 🎬✏️💬📱</h2>
<div class="light-coral-box">
<p class="paragraphs">🎬 Film - und alle anderen Formate, die bewegen.<p>
<p class="paragraphs">Erfahrungen in: </p>
<p class="paragraphs">🎟 Filmmarketing</p>
<p class="paragraphs">🖋 Filmkritik</p>
<p class="paragraphs">📺 Vertrieb und Verleih</p>
<p class="paragraphs">🌎 International und national</p>
<p class="paragraphs">#️⃣🎞 Online und offline</p>
<p class="paragraphs">👩🏻‍🎓 funderiertes kultur- und filmwissenschaftliches Wissen.</p>
</div>

Choose a reason for hiding this comment

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

Something funny 🤡 is going on here with the indentation. Remember that if there's a new block, then you indent one more level. At Technigo, we use 2 spaces for indentation. Also: all the elements that are siblings should be in the same level. 🧑‍🤝‍🧑

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