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

Gt week3 #86

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

Gt week3 #86

wants to merge 8 commits into from

Conversation

GTabala
Copy link

@GTabala GTabala commented Jun 9, 2020

No description provided.

Copy link

@Rody-Kirwan Rody-Kirwan left a comment

Choose a reason for hiding this comment

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

Looks good Gennady.

Left a couple of notes but no need to resubmit. Thanks

<p>Just a small sample of our bakes:</p>
<div id="cake-images-container">
<div class="examples">
<img src="./images/Cakes/cake-with-blackberries.jpg" alt="">

Choose a reason for hiding this comment

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

Should fill in alt values or remove the attribute.

ul {
list-style: none;
text-align: left;
margin-left: 20px;

Choose a reason for hiding this comment

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

The margin-left here is what's causing the horizontal scroll. Use

padding-left: 20px; // sets the space inside the ul element
box-sizing: border-box; // makes sure padding is included in the width

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