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

Restaurant Fiescherblick #57

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

Conversation

Myriam-Schweizer
Copy link

First attemp to create a landingpage for a restaurant. More or less responsive.

I created a first draf for a restaurant website.
Update 30.10.2022
Copy link
Contributor

@HIPPIEKICK HIPPIEKICK 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 with your page! It's not fully responsive, but I think you're on good way! The flexbox is working out well and it's nice that you tried out a lot of different HTML elements and CSS rules. Keep up the good work ⭐

<div class="box">
<div class="worker">
<img src="images/Koch_quadratisch.png" alt="Banana Bread">
<p>Vorname Nachname<br />Position</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

<br/> is not used anymore. You could use flexbox for this ;)

Comment on lines +131 to +132
<div class="footer_menubar">
<div class="footer_menu_item">
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor thing but class name should be named in kebab-case, like this: footer-menu-item

Comment on lines +172 to +175
.gallery1-1:hover{
transform: scale(1.3);
z-index:2;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Very nice hover effect! Just remember indentation

Comment on lines +234 to +260
.box img {
max-width: 200px;
border-radius: 100%;
-webkit-border-radius: 100%;
-moz-border-radius: 100%;
padding: 10px;
margin: 10px;
background: white;
border-color: #04391d;
border: 2px;
border-style: solid;
background-color: #04391d;
}

.box img:hover{
max-width: 200px;
border-radius: 100%;
-webkit-border-radius: 100%;
-moz-border-radius: 100%;
padding: 10px;
margin: 10px;
background: white;
border-color: #04391d;
border: 2px;
border-style: solid;
background-color: #ffffff;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice hover effect here too!
Just remember that you only need to specify the things that are different from you original styling


<body>
<div id="home" class="header">
<a href="./index.html"class="logo"><img src="icons/Fiescherblick Logo grün.svg" alt="Logo Fiescherblick"></a>
Copy link
Contributor

Choose a reason for hiding this comment

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

Really nice that you put your images and icons in different folders. It makes it so much easier to overview your project ⭐

<title>Restaurant Fiescherblick</title>
<!-- dont forget to add a css file and link it here! -->
<link rel="stylesheet" type="text/css" href="style.css">
<!-- If you're using google fonts, that should be added here -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be really nice with a google font 😍


<div class="box2">
<div class="map">
<iframe src="https://www.google.com/maps/embed?pb=!1m14!1m8!1m3!1d10960.292996253223!2d8.0459933!3d46.625315!3m2!1i1024!2i768!4f13.1!3m3!1m2!1s0x0%3A0x4ed11045e4a7d95b!2sHotel%20Fiescherblick!5e0!3m2!1sde!2sch!4v1665255321505!5m2!1sde!2sch" width="700" height="450" style="border:0;" allowfullscreen="" loading="lazy" referrerpolicy="no-referrer-when-downgrade"></iframe>
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

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